lkml.org 
[lkml]   [2011]   [Apr]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] kbuild: Allow to combine multiple W= levels
    Date
    Add support for make W=12, make W=123 and so on, to enable warnings from
    multiple W= levels. Normally, make W=<level> does not include warnings
    from the previous level.

    Signed-off-by: Michal Marek <mmarek@suse.cz>
    ---
    scripts/Makefile.build | 8 ++++++--
    1 files changed, 6 insertions(+), 2 deletions(-)
    diff --git a/scripts/Makefile.build b/scripts/Makefile.build
    index 9c0c481..28cef2a 100644
    --- a/scripts/Makefile.build
    +++ b/scripts/Makefile.build
    @@ -60,6 +60,8 @@ endif
    # $(call cc-option, -W...) handles gcc -W.. options which
    # are not supported by all versions of the compiler
    ifdef KBUILD_ENABLE_EXTRA_GCC_CHECKS
    +warning- := $(empty)
    +
    warning-1 := -Wextra -Wunused -Wno-unused-parameter
    warning-1 += -Wmissing-declarations
    warning-1 += -Wmissing-format-attribute
    @@ -85,9 +87,11 @@ warning-3 += -Wswitch-default
    warning-3 += $(call cc-option, -Wpacked-bitfield-compat)
    warning-3 += $(call cc-option, -Wvla)

    -warning := $(warning-$(KBUILD_ENABLE_EXTRA_GCC_CHECKS))
    +warning := $(warning-$(findstring 1, $(KBUILD_ENABLE_EXTRA_GCC_CHECKS)))
    +warning += $(warning-$(findstring 2, $(KBUILD_ENABLE_EXTRA_GCC_CHECKS)))
    +warning += $(warning-$(findstring 3, $(KBUILD_ENABLE_EXTRA_GCC_CHECKS)))

    -ifeq ("$(warning)","")
    +ifeq ("$(strip $(warning))","")
    $(error W=$(KBUILD_ENABLE_EXTRA_GCC_CHECKS) is unknown)
    endif

    --
    1.7.4.1


    \
     
     \ /
      Last update: 2011-04-29 15:35    [from the cache]
    ©2003-2014 Jasper Spaans. Advertise on this site