On 09/13/2016 05:56 AM, Michael Ellerman wrote: > Shuah Khan <shuahkh@xxxxxxxxxxxxxxx> writes: > >> Update to work under selftests. dnotify_test will not be run as part of >> selftests suite and will not included in install targets. It can be built >> separately for now. >> >> Signed-off-by: Shuah Khan <shuahkh@xxxxxxxxxxxxxxx> >> --- >> tools/testing/selftests/filesystems/Makefile | 10 ++++++---- >> 1 file changed, 6 insertions(+), 4 deletions(-) >> >> diff --git a/tools/testing/selftests/filesystems/Makefile b/tools/testing/selftests/filesystems/Makefile >> index 883010c..f1dce5c 100644 >> --- a/tools/testing/selftests/filesystems/Makefile >> +++ b/tools/testing/selftests/filesystems/Makefile >> @@ -1,5 +1,7 @@ >> -# List of programs to build >> -hostprogs-y := dnotify_test >> +TEST_PROGS := dnotify_test >> +all: $(TEST_PROGS) >> >> -# Tell kbuild to always build the programs >> -always := $(hostprogs-y) >> +include ../lib.mk >> + >> +clean: >> + rm -fr dnotify_test > > That's a complete rewrite of the Makefile, so I don't think there's any > value in bringing its content across from Documentation. Moving Makefile accomplishes delete at the same time. I can combine the move and updating Makefile into one single patch. > > Better IMHO would be to squash this with the previous patch, so we get a > working test under selftests in a single commit. > > cheers > thanks, -- Shuah -- To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
