Re: [PATCH v3 05/20] selftests/ftrace: add install target to enable test install
From: Steven Rostedt <rostedt@goodmis.org>
Date: 2015-01-02 15:45:35
Also in:
linux-api, linux-kbuild, lkml
On Wed, 24 Dec 2014 09:27:41 -0700 Shuah Khan [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Add a new make target to enable installing test. This target installs test in the kselftest install location and add to the kselftest script to run the test. Install target can be run only from top level kernel source directory. Signed-off-by: Shuah Khan <redacted> --- tools/testing/selftests/ftrace/Makefile | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-)diff --git a/tools/testing/selftests/ftrace/Makefile b/tools/testing/selftests/ftrace/Makefile index 76cc9f1..7c7cf42 100644 --- a/tools/testing/selftests/ftrace/Makefile +++ b/tools/testing/selftests/ftrace/Makefile@@ -1,7 +1,16 @@ +TEST_STR = /bin/sh ./ftracetest || echo ftrace selftests: [FAIL]
Is it ok that this removes the quotes around the echo string? I don't see anything wrong about it, but I don't know if there's a shell out there that will fail due to it. Other than than, Acked-by: Steven Rostedt <rostedt@goodmis.org> -- Steve
+ all: +install: +ifdef INSTALL_KSFT_PATH + install ./ftracetest $(INSTALL_KSFT_PATH) + @cp -r test.d $(INSTALL_KSFT_PATH) + echo "$(TEST_STR)" >> $(KSELFTEST) +endif + run_tests: - @/bin/sh ./ftracetest || echo "ftrace selftests: [FAIL]" + @$(TEST_STR) clean: rm -rf logs/*