public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [PATCH] grub-efi: Use a variable to specify built-in grub modules.
@ 2014-04-16  1:16 Philip Tricca
  2014-05-04 22:34 ` Philip Tricca
  0 siblings, 1 reply; 4+ messages in thread
From: Philip Tricca @ 2014-04-16  1:16 UTC (permalink / raw)
  To: openembedded-core

The previous behavior defines a static set of modules that are built
into the grub efi executable. This works fine for a limited set of boot
environments namely the standard linux/initrd. This patch conditionally
 assigns the same modules to a variable. This allows other meta layers
to add additional modules or completely override the defaults. The use
case driving this patch is the use of multiboot2 and related modules.

Signed-off-by: Philip Tricca <flihp@twobit.us>
---
 meta/recipes-bsp/grub/grub-efi_2.00.bb |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-bsp/grub/grub-efi_2.00.bb b/meta/recipes-bsp/grub/grub-efi_2.00.bb
index 6944cb2..4b43749 100644
--- a/meta/recipes-bsp/grub/grub-efi_2.00.bb
+++ b/meta/recipes-bsp/grub/grub-efi_2.00.bb
@@ -70,12 +70,14 @@ do_install_class-native() {
 	install -m 755 grub-mkimage ${D}${bindir}
 }
 
+GRUB_BUILDIN ?= "boot linux ext2 fat serial part_msdos part_gpt normal efi_gop iso9660 search"
+
 do_deploy() {
 	# Search for the grub.cfg on the local boot media by using the
 	# built in cfg file provided via this recipe
 	grub-mkimage -c ../cfg -p /EFI/BOOT -d ./grub-core/ \
 	               -O ${GRUB_TARGET}-efi -o ./${GRUB_IMAGE} \
-	               boot linux ext2 fat serial part_msdos part_gpt normal efi_gop iso9660 search
+	               ${GRUB_BUILDIN}
 	install -m 644 ${B}/${GRUB_IMAGE} ${DEPLOYDIR}
 }
 
-- 
1.7.10.4



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] grub-efi: Use a variable to specify built-in grub modules.
  2014-04-16  1:16 [PATCH] grub-efi: Use a variable to specify built-in grub modules Philip Tricca
@ 2014-05-04 22:34 ` Philip Tricca
  2014-05-05 15:49   ` Saul Wold
  0 siblings, 1 reply; 4+ messages in thread
From: Philip Tricca @ 2014-05-04 22:34 UTC (permalink / raw)
  To: openembedded-core

ping ...

On 04/15/2014 09:16 PM, Philip Tricca wrote:
> The previous behavior defines a static set of modules that are built
> into the grub efi executable. This works fine for a limited set of boot
> environments namely the standard linux/initrd. This patch conditionally
>  assigns the same modules to a variable. This allows other meta layers
> to add additional modules or completely override the defaults. The use
> case driving this patch is the use of multiboot2 and related modules.
> 
> Signed-off-by: Philip Tricca <flihp@twobit.us>
> ---
>  meta/recipes-bsp/grub/grub-efi_2.00.bb |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-bsp/grub/grub-efi_2.00.bb b/meta/recipes-bsp/grub/grub-efi_2.00.bb
> index 6944cb2..4b43749 100644
> --- a/meta/recipes-bsp/grub/grub-efi_2.00.bb
> +++ b/meta/recipes-bsp/grub/grub-efi_2.00.bb
> @@ -70,12 +70,14 @@ do_install_class-native() {
>  	install -m 755 grub-mkimage ${D}${bindir}
>  }
>  
> +GRUB_BUILDIN ?= "boot linux ext2 fat serial part_msdos part_gpt normal efi_gop iso9660 search"
> +
>  do_deploy() {
>  	# Search for the grub.cfg on the local boot media by using the
>  	# built in cfg file provided via this recipe
>  	grub-mkimage -c ../cfg -p /EFI/BOOT -d ./grub-core/ \
>  	               -O ${GRUB_TARGET}-efi -o ./${GRUB_IMAGE} \
> -	               boot linux ext2 fat serial part_msdos part_gpt normal efi_gop iso9660 search
> +	               ${GRUB_BUILDIN}
>  	install -m 644 ${B}/${GRUB_IMAGE} ${DEPLOYDIR}
>  }
>  
> 



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] grub-efi: Use a variable to specify built-in grub modules.
  2014-05-04 22:34 ` Philip Tricca
@ 2014-05-05 15:49   ` Saul Wold
  2014-05-05 16:49     ` Philip Tricca
  0 siblings, 1 reply; 4+ messages in thread
From: Saul Wold @ 2014-05-05 15:49 UTC (permalink / raw)
  To: Philip Tricca, openembedded-core

On 05/04/2014 03:34 PM, Philip Tricca wrote:
> ping ...
>

It was merged on 4/16!

Sau!

> On 04/15/2014 09:16 PM, Philip Tricca wrote:
>> The previous behavior defines a static set of modules that are built
>> into the grub efi executable. This works fine for a limited set of boot
>> environments namely the standard linux/initrd. This patch conditionally
>>   assigns the same modules to a variable. This allows other meta layers
>> to add additional modules or completely override the defaults. The use
>> case driving this patch is the use of multiboot2 and related modules.
>>
>> Signed-off-by: Philip Tricca <flihp@twobit.us>
>> ---
>>   meta/recipes-bsp/grub/grub-efi_2.00.bb |    4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-bsp/grub/grub-efi_2.00.bb b/meta/recipes-bsp/grub/grub-efi_2.00.bb
>> index 6944cb2..4b43749 100644
>> --- a/meta/recipes-bsp/grub/grub-efi_2.00.bb
>> +++ b/meta/recipes-bsp/grub/grub-efi_2.00.bb
>> @@ -70,12 +70,14 @@ do_install_class-native() {
>>   	install -m 755 grub-mkimage ${D}${bindir}
>>   }
>>
>> +GRUB_BUILDIN ?= "boot linux ext2 fat serial part_msdos part_gpt normal efi_gop iso9660 search"
>> +
>>   do_deploy() {
>>   	# Search for the grub.cfg on the local boot media by using the
>>   	# built in cfg file provided via this recipe
>>   	grub-mkimage -c ../cfg -p /EFI/BOOT -d ./grub-core/ \
>>   	               -O ${GRUB_TARGET}-efi -o ./${GRUB_IMAGE} \
>> -	               boot linux ext2 fat serial part_msdos part_gpt normal efi_gop iso9660 search
>> +	               ${GRUB_BUILDIN}
>>   	install -m 644 ${B}/${GRUB_IMAGE} ${DEPLOYDIR}
>>   }
>>
>>
>


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] grub-efi: Use a variable to specify built-in grub modules.
  2014-05-05 15:49   ` Saul Wold
@ 2014-05-05 16:49     ` Philip Tricca
  0 siblings, 0 replies; 4+ messages in thread
From: Philip Tricca @ 2014-05-05 16:49 UTC (permalink / raw)
  To: Saul Wold; +Cc: openembedded-core

On 05/05/2014 11:49 AM, Saul Wold wrote:
> On 05/04/2014 03:34 PM, Philip Tricca wrote:
>> ping ...
>>
> 
> It was merged on 4/16!

Whoops. Was watching the mailing list, not git. My bad.

Thanks!
Philip

> Sau!
> 
>> On 04/15/2014 09:16 PM, Philip Tricca wrote:
>>> The previous behavior defines a static set of modules that are built
>>> into the grub efi executable. This works fine for a limited set of boot
>>> environments namely the standard linux/initrd. This patch conditionally
>>>   assigns the same modules to a variable. This allows other meta layers
>>> to add additional modules or completely override the defaults. The use
>>> case driving this patch is the use of multiboot2 and related modules.
>>>
>>> Signed-off-by: Philip Tricca <flihp@twobit.us>
>>> ---
>>>   meta/recipes-bsp/grub/grub-efi_2.00.bb |    4 +++-
>>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/meta/recipes-bsp/grub/grub-efi_2.00.bb
>>> b/meta/recipes-bsp/grub/grub-efi_2.00.bb
>>> index 6944cb2..4b43749 100644
>>> --- a/meta/recipes-bsp/grub/grub-efi_2.00.bb
>>> +++ b/meta/recipes-bsp/grub/grub-efi_2.00.bb
>>> @@ -70,12 +70,14 @@ do_install_class-native() {
>>>       install -m 755 grub-mkimage ${D}${bindir}
>>>   }
>>>
>>> +GRUB_BUILDIN ?= "boot linux ext2 fat serial part_msdos part_gpt
>>> normal efi_gop iso9660 search"
>>> +
>>>   do_deploy() {
>>>       # Search for the grub.cfg on the local boot media by using the
>>>       # built in cfg file provided via this recipe
>>>       grub-mkimage -c ../cfg -p /EFI/BOOT -d ./grub-core/ \
>>>                      -O ${GRUB_TARGET}-efi -o ./${GRUB_IMAGE} \
>>> -                   boot linux ext2 fat serial part_msdos part_gpt
>>> normal efi_gop iso9660 search
>>> +                   ${GRUB_BUILDIN}
>>>       install -m 644 ${B}/${GRUB_IMAGE} ${DEPLOYDIR}
>>>   }
>>>
>>>
>>



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-05-05 16:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-16  1:16 [PATCH] grub-efi: Use a variable to specify built-in grub modules Philip Tricca
2014-05-04 22:34 ` Philip Tricca
2014-05-05 15:49   ` Saul Wold
2014-05-05 16:49     ` Philip Tricca

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox