[Buildroot] [PATCH v4 2/2] apply-patches.sh: don't print anything when "make -s" is used
fabio.porcedda at gmail.com
Thu Aug 7 07:37:13 UTC 2014
On Wed, Aug 6, 2014 at 9:16 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> Fabio, All,
> On 2014-08-01 13:24 +0200, Fabio Porcedda spake thusly:
>> The make "-s" option is used to enable the "Silent operation" so if that
>> option is used don't print anything as far as there isn't any error.
>> Add the "-s" option to "apply-patches.sh" to enable silent operation.
>> Also add the "BR_SILENT" variable the contain "YES" when "make -s" is
>> used so others parts can use it to silence the build as well.
>> Signed-off-by: Fabio Porcedda <fabio.porcedda at gmail.com>
> Reviewed-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> HOwever, I have a comment about it, see below...
>> Makefile | 2 ++
>> package/Makefile.in | 2 +-
>> support/scripts/apply-patches.sh | 17 ++++++++++++++---
>> 3 files changed, 17 insertions(+), 4 deletions(-)
>> diff --git a/Makefile b/Makefile
>> index c82b07f..73eb033 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -198,6 +198,8 @@ else
>> Q = @
>> +BR_SILENT = $(if $(findstring s,$(MAKEFLAGS)),YES)
> Unfortunately, that also matches '--warn-undefined-variables' since it
> has an 's' in it. It is the onlt long option with an 's' in it, for
> which there is no corresponding short option. All other long options
> with an 's' in them have corresponding short options, and that's what
> appears in MAKEFLAGS.
Good point, I was confused by the official documentation of Make:
Using the filter function instead of substring fix the problem:
-BR_SILENT = $(if $(findstring s,$(MAKEFLAGS)),YES)
+BR_SILENT = $(if $(filter s -s,$(MAKEFLAGS)),YES)
I will send an updated patch.
> I think we can well live with the fact that --warn-undefined-variables
> would trigger a silent patching, because this should really only be
> rarely used, if at all, because we do rely on variables not being
> defined, so this is not an error, not even a warning.
> That's why I added my reviewed tag.
More information about the buildroot