Re: [PATCH 4/8] selftests: gpio: override clean in lib.mk to fix warnings
On 04/21/2017 11:40 PM, Michael Ellerman wrote: > Shuah Khanwrites: > >> Add override for lib.mk clean to fix the following warnings from clean >> target run. >> >> Makefile:11: warning: overriding recipe for target 'clean' >> ../lib.mk:55: warning: ignoring old recipe for target 'clean' >> >> Signed-off-by: Shuah Khan >> --- >> tools/testing/selftests/gpio/Makefile | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/tools/testing/selftests/gpio/Makefile >> b/tools/testing/selftests/gpio/Makefile >> index 205e4d1..4f6d9e0 100644 >> --- a/tools/testing/selftests/gpio/Makefile >> +++ b/tools/testing/selftests/gpio/Makefile >> @@ -7,8 +7,9 @@ include ../lib.mk >> >> all: $(BINARIES) >> >> -clean: >> +override define CLEAN >> $(RM) $(BINARIES) >> +endef > > This could be achieved more simply with: > > EXTRA_CLEAN := $(BINARIES) > gpio clean requires special handling. I have one more patch I sent out that handles that. So I am going to leave this patch the way with override. thanks, -- Shuah
Re: [PATCH 4/8] selftests: gpio: override clean in lib.mk to fix warnings
On 04/21/2017 11:40 PM, Michael Ellerman wrote: > Shuah Khan writes: > >> Add override for lib.mk clean to fix the following warnings from clean >> target run. >> >> Makefile:11: warning: overriding recipe for target 'clean' >> ../lib.mk:55: warning: ignoring old recipe for target 'clean' >> >> Signed-off-by: Shuah Khan >> --- >> tools/testing/selftests/gpio/Makefile | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/tools/testing/selftests/gpio/Makefile >> b/tools/testing/selftests/gpio/Makefile >> index 205e4d1..4f6d9e0 100644 >> --- a/tools/testing/selftests/gpio/Makefile >> +++ b/tools/testing/selftests/gpio/Makefile >> @@ -7,8 +7,9 @@ include ../lib.mk >> >> all: $(BINARIES) >> >> -clean: >> +override define CLEAN >> $(RM) $(BINARIES) >> +endef > > This could be achieved more simply with: > > EXTRA_CLEAN := $(BINARIES) > gpio clean requires special handling. I have one more patch I sent out that handles that. So I am going to leave this patch the way with override. thanks, -- Shuah
Re: [PATCH 4/8] selftests: gpio: override clean in lib.mk to fix warnings
Shuah Khanwrites: > Add override for lib.mk clean to fix the following warnings from clean > target run. > > Makefile:11: warning: overriding recipe for target 'clean' > ../lib.mk:55: warning: ignoring old recipe for target 'clean' > > Signed-off-by: Shuah Khan > --- > tools/testing/selftests/gpio/Makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/gpio/Makefile > b/tools/testing/selftests/gpio/Makefile > index 205e4d1..4f6d9e0 100644 > --- a/tools/testing/selftests/gpio/Makefile > +++ b/tools/testing/selftests/gpio/Makefile > @@ -7,8 +7,9 @@ include ../lib.mk > > all: $(BINARIES) > > -clean: > +override define CLEAN > $(RM) $(BINARIES) > +endef This could be achieved more simply with: EXTRA_CLEAN := $(BINARIES) cheers
Re: [PATCH 4/8] selftests: gpio: override clean in lib.mk to fix warnings
Shuah Khan writes: > Add override for lib.mk clean to fix the following warnings from clean > target run. > > Makefile:11: warning: overriding recipe for target 'clean' > ../lib.mk:55: warning: ignoring old recipe for target 'clean' > > Signed-off-by: Shuah Khan > --- > tools/testing/selftests/gpio/Makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/gpio/Makefile > b/tools/testing/selftests/gpio/Makefile > index 205e4d1..4f6d9e0 100644 > --- a/tools/testing/selftests/gpio/Makefile > +++ b/tools/testing/selftests/gpio/Makefile > @@ -7,8 +7,9 @@ include ../lib.mk > > all: $(BINARIES) > > -clean: > +override define CLEAN > $(RM) $(BINARIES) > +endef This could be achieved more simply with: EXTRA_CLEAN := $(BINARIES) cheers
[PATCH 4/8] selftests: gpio: override clean in lib.mk to fix warnings
Add override for lib.mk clean to fix the following warnings from clean target run. Makefile:11: warning: overriding recipe for target 'clean' ../lib.mk:55: warning: ignoring old recipe for target 'clean' Signed-off-by: Shuah Khan--- tools/testing/selftests/gpio/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/gpio/Makefile b/tools/testing/selftests/gpio/Makefile index 205e4d1..4f6d9e0 100644 --- a/tools/testing/selftests/gpio/Makefile +++ b/tools/testing/selftests/gpio/Makefile @@ -7,8 +7,9 @@ include ../lib.mk all: $(BINARIES) -clean: +override define CLEAN $(RM) $(BINARIES) +endef CFLAGS += -O2 -g -std=gnu99 -Wall -I../../../../usr/include/ LDLIBS += -lmount -I/usr/include/libmount -- 2.9.3
[PATCH 4/8] selftests: gpio: override clean in lib.mk to fix warnings
Add override for lib.mk clean to fix the following warnings from clean target run. Makefile:11: warning: overriding recipe for target 'clean' ../lib.mk:55: warning: ignoring old recipe for target 'clean' Signed-off-by: Shuah Khan --- tools/testing/selftests/gpio/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/gpio/Makefile b/tools/testing/selftests/gpio/Makefile index 205e4d1..4f6d9e0 100644 --- a/tools/testing/selftests/gpio/Makefile +++ b/tools/testing/selftests/gpio/Makefile @@ -7,8 +7,9 @@ include ../lib.mk all: $(BINARIES) -clean: +override define CLEAN $(RM) $(BINARIES) +endef CFLAGS += -O2 -g -std=gnu99 -Wall -I../../../../usr/include/ LDLIBS += -lmount -I/usr/include/libmount -- 2.9.3