[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-core
Subject:    Re: [OE-core] [PATCH] systemd: check 'efi' in MACHINE_FEATURES
From:       Mikko Ylinen <mikko.ylinen () linux ! intel ! com>
Date:       2017-03-31 12:00:02
Message-ID: 4210425e-9602-9573-e3e6-43a515a9b7d1 () linux ! intel ! com
[Download RAW message or body]


On 30/03/17 19:15, Burton, Ross wrote:
>
> On 30 March 2017 at 07:59, Mikko Ylinen <mikko.ylinen@linux.intel.com 
> <mailto:mikko.ylinen@linux.intel.com>> wrote:
>
>     I'm fine. Perhaps 'efi' should not be a PACKAGECONFIG at all (and
>     unconditionally disabled here) and
>     the functionality (just bootctl?) is moved in systemd-boot. Thoughts?
>
>
> I'm not fully aware of the details here but this seems like a good 
> idea.  Can you prepare a patch?
>

Looks like it's not very easy to build bootctl part of systemd-boot 
(without re-building the whole
systemd there too).

Therefore, I'd just:

-                   ${@bb.utils.filter('DISTRO_FEATURES', 'efi pam 
selinux ldconfig', d)} \
+                   ${@bb.utils.filter('DISTRO_FEATURES', 'pam selinux 
ldconfig', d)} \

and perhaps:

-PACKAGECONFIG[efi] = "--enable-efi,--disable-efi"
+PACKAGECONFIG[bootctl] = "--enable-efi,--disable-efi"

efi/bootctl can then be added in PACKAGECONFIG by those who need bootctl.

-- Mikko
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
[prev in list] [next in list] [prev in thread] [next in thread]