[OE-core] [PATCH v2 1/9] uefi.bbclass: add bbclass holding configuration for UEFI applications

[Dmitry Eremin-Solenikov]

вт, 17 сент. 2019 г. в 01:17, Richard Purdie
<richard.purdie at linuxfoundation.org>:
>
> On Fri, 2019-09-13 at 18:44 +0300, dbaryshkov at gmail.com wrote:
> > From: Dmitry Eremin-Solenikov <dmitry_eremin-solenikov at mentor.com>
> >
> > Create new bbclass defining common variables for all UEFI-related
> > packages (bootloaders, test applications, etc).
> >
> > Signed-off-by: Dmitry Eremin-Solenikov <
> > dmitry_eremin-solenikov at mentor.com>
> > ---
> >  meta/classes/uefi.bbclass | 26 ++++++++++++++++++++++++++
> >  1 file changed, 26 insertions(+)
> >  create mode 100644 meta/classes/uefi.bbclass
>
> I really want to get away from the proliferation of bbclass files we
> have. Wouldn't this make more sense as a .conf file?

Moving configuration to .conf file might make sense. I even can
implement anonymous function as per-arch override. However there are
still functions like efi_populate_common() and
efi_iso_populate_common(), which can not be moved to .conf file.

Would you prefer separate uefi.conf file and uefi-bootloader.bbclass files?

> The anonymous function could be done with overrides, or an appropriate
> function from lib/oe/.
>
> I appreciate we don't use conf files so much but we need to start
> somewhere and I think this is a good candidate for it.

-- 
With best wishes
Dmitry