[Buildroot] [PATCH v3 2/7] boot/grub2: add support to build multiple Grub2 configurations in the same build
Yann E. MORIN
yann.morin.1998 at free.fr
Mon Sep 27 19:42:50 UTC 2021
Köry, All,
On 2021-09-23 17:57 +0200, Kory Maincent spake thusly:
> This patch allows to build Grub2 for different configurations simultaneously.
> To cover Grub2 configuration of legacy BIOS platforms (32-bit), 32-bit EFI
> BIOS and 64-bit EFI BIOS in the same build, multi-build system felt much more
> reasonable to just extend the grub2 package into 3 packages.
[--SNIP--]
> diff --git a/Config.in.legacy b/Config.in.legacy
> index 35a11f4dc6..33cee0202b 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -168,6 +168,30 @@ config BR2_KERNEL_HEADERS_5_12
>
> comment "Legacy options removed in 2021.08"
>
> +config BR2_TARGET_GRUB2_BUILTIN_MODULES
> + string "the grub2 builtin modules has been renamed"
> + help
> + This option has been split to separate the builtin modules
> + between BR2_TARGET_GRUB2_BUILTIN_MODULES_PC and
> + BR2_TARGET_GRUB2_BUILTIN_MODULES_EFI.
> +
> +config BR2_TARGET_GRUB2_BUILTIN_MODULES_WRAP
> + bool
> + default y if BR2_TARGET_GRUB2_BUILTIN_MODULES != ""
> + select BR2_LEGACY
> +
> +config BR2_TARGET_GRUB2_BUILTIN_CONFIG
> + string "the grub2 builtin configuration has been renamed"
> + help
> + This option has been split to separate the builtin configuration
$ make check-package
Config.in.legacy:186: help text: <tab><2 spaces><62 chars>
(http://nightly.buildroot.org/#writing-rules-config-in)
[--SNIP--]
> diff --git a/boot/grub2/Config.in b/boot/grub2/Config.in
> index e45133999e..10bba6c5e4 100644
> --- a/boot/grub2/Config.in
> +++ b/boot/grub2/Config.in
> @@ -10,6 +10,13 @@ config BR2_TARGET_GRUB2
> bool "grub2"
> depends on BR2_TARGET_GRUB2_ARCH_SUPPORTS
> depends on BR2_USE_WCHAR
> + select BR2_TARGET_GRUB2_I386_PC if \
> + !BR2_TARGET_GRUB2_HAS_PTF && \
> + (BR2_i386 || BR2_x86_64)
> + select BR2_TARGET_GRUB2_ARM_UBOOT if \
> + !BR2_TARGET_GRUB2_HAS_PTF && \
> + BR2_arm
> + select BR2_TARGET_GRUB2_ARM64_EFI if BR2_aarch64
> help
> GNU GRUB is a Multiboot boot loader. It was derived from
> GRUB, the GRand Unified Bootloader, which was originally
> @@ -25,10 +32,10 @@ config BR2_TARGET_GRUB2
>
> http://www.gnu.org/software/grub/
>
> -if BR2_TARGET_GRUB2
> +config BR2_TARGET_GRUB2_HAS_PTF
> + bool
I've moved this option inside the 'if' conditional, otherwise the
suboptions are not properly indented.
Applied to master with the above two fixed, thanks.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list