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

[Anders Darander <anders@chargestorm.se>]

* Bruno Bottazzini <bruno.bottazzini@intel.com> [150511 20:03]:

> diff --git a/meta/recipes-core/systemd/systemd_219.bb b/meta/recipes-core/systemd/systemd_219.bb
> index 6ed4fe0..e83601f 100644
> --- a/meta/recipes-core/systemd/systemd_219.bb
> +++ b/meta/recipes-core/systemd/systemd_219.bb
> @@ -19,7 +19,18 @@ PROVIDES = "udev"

>  PE = "1"

> -DEPENDS = "kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl readline dbus libcap libcgroup glib-2.0 qemu-native util-linux"
> +DEPENDS = "intltool-native \
> +           gperf-native \
> +           readline \
> +           libcgroup \
> +           libcap \
> +           qemu-native \
> +           ${@bb.utils.contains('PACKAGECONFIG', 'dbus', 'dbus', '', d)} \
> +           ${@bb.utils.contains('PACKAGECONFIG', 'kmod', 'kmod', '', d)} \
> +           ${@bb.utils.contains('PACKAGECONFIG', 'manpages', 'docbook-sgml-dtd-4.1-native', '', d)} \
> +           ${@bb.utils.contains('PACKAGECONFIG', 'acl', 'acl', '', d)} \
> +           ${@bb.utils.contains('PACKAGECONFIG', 'glib', 'glib-2.0', '', d)} \
> +           ${@bb.utils.contains('PACKAGECONFIG', 'blkid', 'util-linux', '', d)}"

These lines should be removed. The 3rd parameter in the PACKAGECONFIG[]
definitions below, do automatically get added to DEPENDS if the
respective PACKAGECONFIG is enabled. Thus, you don't need to check for
it yourself in DEPENDS.

Cheers,
Anders

-- 
Anders Darander
ChargeStorm AB / eStorm AB