[OE-core] [PATCH 3/3] systemd: split modules into packages

[Bottazzini, Bruno]

On Qui, 2015-03-05 at 15:28 +0100, Anders Darander wrote:
> * Bruno Bottazzini <bruno.bottazzini at intel.com> [150304 22:28]:
> > if one wants to launch a simple deamon, most modules are not
> > required.
> > He will be able to save space and exclude unwanted packages
> > from the final image.
> > ---
> >  meta/recipes-core/systemd/systemd_219.bb | 1071 ++++++++++++++++++++++++++----
> >  1 file changed, 942 insertions(+), 129 deletions(-)
> 
> > diff --git a/meta/recipes-core/systemd/systemd_219.bb b/meta/recipes-core/systemd/systemd_219.bb
> > index 6a8b5b8..fd6cf57 100644
> > --- a/meta/recipes-core/systemd/systemd_219.bb
> > +++ b/meta/recipes-core/systemd/systemd_219.bb
> > @@ -56,10 +56,31 @@ LDFLAGS_append_libc-uclibc = " -lrt"
> 
> >  GTKDOC_DOCDIR = "${S}/docs/"
> 
> > -PACKAGECONFIG ??= "xz \
> > +DEPENDS += "curl"
> > +
> > +PACKAGECONFIG ??= "acl \
> > +                   blkid \
> > +                   efi \
> > +                   kmod \
> > +                   gcrypt \
> > +                   lz4 \
> > +                   xz \
> > +                   libidn \
> >                     ${@bb.utils.contains('DISTRO_FEATURES', 'pam', 'pam', '', d)} \
> >                     ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'xkbcommon', '', d)}"
> 
> Just a quick question before I look into the patch in more detail.
> 
> Is the new setting of PACKAGECONFIG consistent with how systemd was
> built previously? I guess it is.

Hi Anders,

it is consistent with how systemd was built previously. If you apply the
patch and bitbake it. Systemd will be built and shipped normally.

But now it will give some options on how to customize it by excluding
packages you don't want to be with systemd.

> 
> Another comment, you should remove the dependcies that gets added using
> PACKAGECONFIG from DEPENDS, e.g. acl etc. (Or are they required
> nevertheless?)

You mean I should do the following ?
- PACKAGECONFIG[acl] = "--enable-acl,--disable-acl,acl"
+ PACKAGECONFIG[acl] = "--enable-acl,--disable-acl"

If I get what you said correctly, yes they are required. 

Best Regards,

> 
> Cheers,
> Anders
>