diff options
author | Enrico Jörns <ejo@pengutronix.de> | 2025-02-28 08:42:27 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2025-03-03 18:01:29 +0000 |
commit | 44139c8c69f66400f94a449104a83c10a3935b94 (patch) | |
tree | 58e1387576b8780a4c49508ac6a05bf0e2143b44 /meta/classes-recipe | |
parent | 4dfed23581ab25f032356bbf32f08c87d01ec598 (diff) | |
download | poky-44139c8c69f66400f94a449104a83c10a3935b94.tar.gz |
cml1.bbclass: use consistent make flags for menuconfig
The class called 'make menuconfig' without any of the make variables and
options set in EXTRA_OEMAKE, resulting in a quite different build
environment than actually intended.
For the kernel.bbclass this was fixed in commit 8c616bc0 ("kernel: Use
consistent make flags for menuconfig") by appending ${EXTRA_OEMAKE} to
KCONFIG_CONFIG_COMMAND.
Instead of fixing this individually for additional recipes, we simply
include ${EXTRA_OEMAKE} in KCONFIG_CONFIG_COMMAND by default.
For most class users, this change is directly visible in the generated
.config file:
* For barebox and u-boot, the CONFIG_GCC_VERSION erroneously reflected
the host GCC version before where it now correctly reflects the target
toolchain's GCC.
* For u-boot, also the "Compiler: " line at the beginning of the .config
now prints the target toolchain instead of the host ones.
* The kernel had this already set.
* busybox did not produce any difference.
Note that these projects might base some compile-time decisions on e.g.
the actual compiler version used. Having the wrong one in the
menuconfig-generated .config affects at least the visibility and
consistency.
Reported-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
(From OE-Core rev: 1b6ddd452837e67b500a84455a234f5edc8250a9)
Signed-off-by: Enrico Jörns <ejo@pengutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes-recipe')
-rw-r--r-- | meta/classes-recipe/cml1.bbclass | 2 | ||||
-rw-r--r-- | meta/classes-recipe/kernel.bbclass | 3 |
2 files changed, 1 insertions, 4 deletions
diff --git a/meta/classes-recipe/cml1.bbclass b/meta/classes-recipe/cml1.bbclass index d18b008aec..3c2b4da4af 100644 --- a/meta/classes-recipe/cml1.bbclass +++ b/meta/classes-recipe/cml1.bbclass | |||
@@ -31,7 +31,7 @@ CROSS_CURSES_LIB = "-lncurses -ltinfo" | |||
31 | CROSS_CURSES_INC = '-DCURSES_LOC="<curses.h>"' | 31 | CROSS_CURSES_INC = '-DCURSES_LOC="<curses.h>"' |
32 | TERMINFO = "${STAGING_DATADIR_NATIVE}/terminfo" | 32 | TERMINFO = "${STAGING_DATADIR_NATIVE}/terminfo" |
33 | 33 | ||
34 | KCONFIG_CONFIG_COMMAND ??= "menuconfig" | 34 | KCONFIG_CONFIG_COMMAND ??= "menuconfig ${EXTRA_OEMAKE}" |
35 | KCONFIG_CONFIG_ENABLE_MENUCONFIG ??= "true" | 35 | KCONFIG_CONFIG_ENABLE_MENUCONFIG ??= "true" |
36 | KCONFIG_CONFIG_ROOTDIR ??= "${B}" | 36 | KCONFIG_CONFIG_ROOTDIR ??= "${B}" |
37 | python do_menuconfig() { | 37 | python do_menuconfig() { |
diff --git a/meta/classes-recipe/kernel.bbclass b/meta/classes-recipe/kernel.bbclass index 6fe7c60e17..64a685a964 100644 --- a/meta/classes-recipe/kernel.bbclass +++ b/meta/classes-recipe/kernel.bbclass | |||
@@ -691,9 +691,6 @@ kernel_do_configure() { | |||
691 | 691 | ||
692 | inherit cml1 pkgconfig | 692 | inherit cml1 pkgconfig |
693 | 693 | ||
694 | # Need LD, HOSTLDFLAGS and more for config operations | ||
695 | KCONFIG_CONFIG_COMMAND:append = " ${EXTRA_OEMAKE}" | ||
696 | |||
697 | EXPORT_FUNCTIONS do_compile do_transform_kernel do_transform_bundled_initramfs do_install do_configure | 694 | EXPORT_FUNCTIONS do_compile do_transform_kernel do_transform_bundled_initramfs do_install do_configure |
698 | 695 | ||
699 | # kernel-base becomes kernel-${KERNEL_VERSION} | 696 | # kernel-base becomes kernel-${KERNEL_VERSION} |