[Buildroot] [PATCH v4] Makefile: improve detection of make "-s" flag
fabio.porcedda at gmail.com
Tue Dec 9 17:48:33 UTC 2014
On Wed, Dec 3, 2014 at 7:17 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> Fabio, All,
> On 2014-11-19 18:51 +0100, Fabio Porcedda spake thusly:
>> Because it's just checked the presence of the "s" character even a
>> make --warn-undefined-variables
>> is detected as a silent build so fix this by filtering out long options.
>> Signed-off-by: Fabio Porcedda <fabio.porcedda at gmail.com>
>> Cc: Arnout Vandecappelle <arnout at mind.be>
>> - split this patch from the patch set to send to muster as bugfix
>> - remove spurious space at the beginning of the QUIET variable (Arnout)
>> Makefile | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> diff --git a/Makefile b/Makefile
>> index 471edf9..6b97dc6 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -303,7 +303,7 @@ GNU_HOST_NAME := $(shell support/gnuconfig/config.guess)
>> TARGETS :=
>> # silent mode requested?
>> -QUIET := $(if $(findstring s,$(MAKEFLAGS)),-q)
>> +QUIET := $(if $(findstring s, $(filter-out --%, $(MAKEFLAGS))),-q)
> Please, do not add a space after the comma when calling a macro. This
> should be:
> QUIET := $(if $(findstring s,$(filter-out --%,$(MAKEFLAGS))),-q)
> Yes, that's slightly less readable. But make really cuts on commas, and
> the sapce is then part of the values, so we do not want it (it has no
> impact in your case, though, but: consistency! ;-) )
Ok done, i will push an updated path set.
> Otherwise, looks good to me.
> Yann E. MORIN.
More information about the buildroot