All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] grub: split grub-editenv into it's own package
@ 2016-08-12 13:15 Ioan-Adrian Ratiu
  2016-08-16 13:18 ` Burton, Ross
  0 siblings, 1 reply; 3+ messages in thread
From: Ioan-Adrian Ratiu @ 2016-08-12 13:15 UTC (permalink / raw)
  To: openembedded-core

From: Alejandro del Castillo <alejandro.delcastillo@ni.com>

grub-editenv edits the env block at runtime on a booted system. Other
tools can depend on it to configure a live system, for ex. to set next
boot mode upon reboot. By splitting grub-editenv, tools don't have to
depend on the entire grub package (grub-editenv just edits one file).

Signed-off-by: Alejandro del Castillo <alejandro.delcastillo@ni.com>
Signed-off-by: Ioan-Adrian Ratiu <adrian.ratiu@ni.com>
---
 meta/recipes-bsp/grub/grub_2.00.bb | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-bsp/grub/grub_2.00.bb b/meta/recipes-bsp/grub/grub_2.00.bb
index 778074a..f983e77 100644
--- a/meta/recipes-bsp/grub/grub_2.00.bb
+++ b/meta/recipes-bsp/grub/grub_2.00.bb
@@ -8,6 +8,10 @@ EXTRA_OECONF = "--with-platform=pc --disable-grub-mkfont --program-prefix="" \
 
 EXTRA_OECONF += "${@bb.utils.contains('DISTRO_FEATURES', 'largefile', '--enable-largefile', '--disable-largefile', d)}"
 
+PACKAGES =+ "grub-editenv"
+
+FILES_grub-editenv = "${bindir}/grub-editenv"
+
 do_install_append () {
     install -d ${D}${sysconfdir}/grub.d
 }
-- 
2.9.2



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

* Re: [PATCH] grub: split grub-editenv into it's own package
  2016-08-12 13:15 [PATCH] grub: split grub-editenv into it's own package Ioan-Adrian Ratiu
@ 2016-08-16 13:18 ` Burton, Ross
  2016-08-16 13:39   ` Ioan-Adrian Ratiu
  0 siblings, 1 reply; 3+ messages in thread
From: Burton, Ross @ 2016-08-16 13:18 UTC (permalink / raw)
  To: Ioan-Adrian Ratiu; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 240 bytes --]

On 12 August 2016 at 14:15, Ioan-Adrian Ratiu <adrian.ratiu@ni.com> wrote:

> +PACKAGES =+ "grub-editenv"
> +
> +FILES_grub-editenv = "${bindir}/grub-editenv"
>

Shouldn't grub rdepend, or at least rrecommend, grub-editenv?

Ross

[-- Attachment #2: Type: text/html, Size: 717 bytes --]

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

* Re: [PATCH] grub: split grub-editenv into it's own package
  2016-08-16 13:18 ` Burton, Ross
@ 2016-08-16 13:39   ` Ioan-Adrian Ratiu
  0 siblings, 0 replies; 3+ messages in thread
From: Ioan-Adrian Ratiu @ 2016-08-16 13:39 UTC (permalink / raw)
  To: Burton, Ross; +Cc: OE-core

On Tue, 16 Aug 2016, "Burton, Ross" <ross.burton@intel.com> wrote:
> On 12 August 2016 at 14:15, Ioan-Adrian Ratiu <adrian.ratiu@ni.com> wrote:
>
>> +PACKAGES =+ "grub-editenv"
>> +
>> +FILES_grub-editenv = "${bindir}/grub-editenv"
>>
>
> Shouldn't grub rdepend, or at least rrecommend, grub-editenv?

Good catch. RDEPENDS_grub would preserve the previous behaviour.
I'll send v2.

Thank you!

>
> Ross


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

end of thread, other threads:[~2016-08-16 13:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-12 13:15 [PATCH] grub: split grub-editenv into it's own package Ioan-Adrian Ratiu
2016-08-16 13:18 ` Burton, Ross
2016-08-16 13:39   ` Ioan-Adrian Ratiu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.