Re: [PATCH 2/9] selftests: update filesystems Makefile to work under selftests
From: Shuah Khan
Date: Tue Sep 13 2016 - 09:21:22 EST
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
>> + 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.