Re: [PATCH 3/8] selftests: futex: override clean in lib.mk to fix warnings

From: Shuah Khan
Date: Thu Apr 27 2017 - 17:57:31 EST


On 04/27/2017 03:54 PM, Darren Hart wrote:
> On Fri, Apr 21, 2017 at 05:14:45PM -0600, Shuah Khan wrote:
>> Add override for lib.mk clean to fix the following warnings from clean
>> target run.
>>
>> Makefile:36: warning: overriding recipe for target 'clean'
>> ../lib.mk:55: warning: ignoring old recipe for target 'clean'
>>
>> Signed-off-by: Shuah Khan <shuahkh@xxxxxxxxxxxxxxx>
>> ---
>> tools/testing/selftests/futex/Makefile | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/tools/testing/selftests/futex/Makefile b/tools/testing/selftests/futex/Makefile
>> index c8095e6..e2fbb89 100644
>> --- a/tools/testing/selftests/futex/Makefile
>> +++ b/tools/testing/selftests/futex/Makefile
>> @@ -32,9 +32,10 @@ override define EMIT_TESTS
>> echo "./run.sh"
>> endef
>>
>> -clean:
>> +override define CLEAN
>> for DIR in $(SUBDIRS); do \
>> BUILD_TARGET=$(OUTPUT)/$$DIR; \
>> mkdir $$BUILD_TARGET -p; \
>> make OUTPUT=$$BUILD_TARGET -C $$DIR $@;\
>> done
>> +endef
>
> Taking the move of clean into lib.mk as a given,

Yeah I considered undoing that, and chose to fix the missed
issues instead.
>
> Acked-by: Darren Hart (VMware) <dvhart@xxxxxxxxxxxxx>
>

thanks,
-- Shuah