Re: [PATCH 3/3] systemd: split modules into packages

[Anders Darander <anders@chargestorm.se>]

* Bruno Bottazzini <bruno.bottazzini@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.

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

Cheers,
Anders

-- 
Anders Darander
ChargeStorm AB / eStorm AB