diff options
author | Daniel Golle <daniel@makrotopia.org> | 2024-04-07 19:50:04 +0100 |
---|---|---|
committer | Daniel Golle <daniel@makrotopia.org> | 2024-04-08 01:40:15 +0100 |
commit | 501ef81040baa2ee31de6dd9f75d619de0e4c9bc (patch) | |
tree | b5b24ca9dc9c402651b7c4febd2e565a726d6105 | |
parent | 5e0587b8d1274096620031951f0fe0449913740d (diff) | |
download | openwrt-501ef81040baa2ee31de6dd9f75d619de0e4c9bc.tar.gz openwrt-501ef81040baa2ee31de6dd9f75d619de0e4c9bc.tar.bz2 openwrt-501ef81040baa2ee31de6dd9f75d619de0e4c9bc.zip |
config: select KERNEL_WERROR if building with default GCC version
At the moment we have to manually follow the default GCC version
also in config/Config-kernel.in. This tends to be forgotten at GCC
version bumps (just happened when switching from version 12 to 13).
Instead, introduce a hidden Kconfig symbol which implies KERNEL_WERROR
in toolchain/gcc/Config.in where it is visible for developers changing
the default version.
Also remove the explicit default on BUILDBOT to avoid a circular
dependency and also because buildbots anyway implicitly always select
the default GCC version.
Reference: https://github.com/openwrt/openwrt/pull/15064
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
-rw-r--r-- | config/Config-kernel.in | 2 | ||||
-rw-r--r-- | toolchain/gcc/Config.in | 5 |
2 files changed, 5 insertions, 2 deletions
diff --git a/config/Config-kernel.in b/config/Config-kernel.in index f4ea6ccbe8..feabf0870e 100644 --- a/config/Config-kernel.in +++ b/config/Config-kernel.in @@ -1418,8 +1418,6 @@ config KERNEL_JFFS2_FS_SECURITY config KERNEL_WERROR bool "Compile the kernel with warnings as errors" - default BUILDBOT - default y if GCC_USE_VERSION_12 help A kernel build should not cause any compiler warnings, and this enables the '-Werror' (for C) and '-Dwarnings' (for Rust) flags diff --git a/toolchain/gcc/Config.in b/toolchain/gcc/Config.in index 85abbdabb7..fd98914075 100644 --- a/toolchain/gcc/Config.in +++ b/toolchain/gcc/Config.in @@ -16,6 +16,11 @@ choice bool "gcc 13.x" endchoice +config GCC_USE_DEFAULT_VERSION + bool + default y if !TOOLCHAINOPTS || GCC_USE_VERSION_13 + imply KERNEL_WERROR + config GCC_USE_GRAPHITE bool prompt "Compile in support for the new Graphite framework in GCC 4.4+" if TOOLCHAINOPTS |