Re: [PATCH 0/2] Initial ergonomic improvements to scripts/Kconfig.toolchain

[Nicolas Schier]

> Hi all,
> 
> This series includes a cleanup to scripts/Kconfig.toolchain around
> dependencies (the first patch) and improves the ergonomics of using
> cc-option and cc-option-str in Kconfig to disable warnings, making
> Kconfig and Kbuild more unified in their behavior (the second patch).
> 
> ---
> Nathan Chancellor (2):
>       scripts/Kconfig.toolchain: Drop compiler symbols from dependencies
>       kconfig: Improve ergonomics around disabling warnings with cc-option
> 
>  scripts/Kconfig.include   |  2 +-
>  scripts/Kconfig.toolchain | 22 +++++++++++-----------
>  2 files changed, 12 insertions(+), 12 deletions(-)

Thanks, straight-forward!

When I look into the resulting .config, I'd wish, these empty string
kconfig symbols could be unset, e.g.

    # CONFIG_CC_OPT_WNO_FORMAT_TRUNCATION_NON_KPRINTF is not set
    # CONFIG_CC_OPT_WNO_DEFAULT_CONST_INIT_UNSAFE is not set
    CONFIG_CC_OPT_WNO_DANGLING_POINTER="-Wno-dangling-pointer"
    CONFIG_CC_OPT_WVLA_LARGER_THAN="-Wvla-larger-than=1"
    CONFIG_CC_HAS_WSTRINGOP_OVERFLOW=y

instead of the current:

    CONFIG_CC_OPT_WNO_FORMAT_TRUNCATION_NON_KPRINTF=""
    CONFIG_CC_OPT_WNO_DEFAULT_CONST_INIT_UNSAFE=""
    CONFIG_CC_OPT_WNO_DANGLING_POINTER="-Wno-dangling-pointer"
    CONFIG_CC_OPT_WVLA_LARGER_THAN="-Wvla-larger-than=1"
    CONFIG_CC_HAS_WSTRINGOP_OVERFLOW=y

but that would make things unneccessary more complicated, right now.

Reviewed-by: Nicolas Schier <nsc@xxxxxxxxxx>

-- 
Nicolas