[OE-core] [PATCH v2] connman: fix systemd support for connman-* packages.

Burton, Ross ross.burton at intel.com
Tue Aug 20 11:58:51 UTC 2013


On 13 August 2013 12:26, Yevhen Kyriukha <kirgene at gmail.com> wrote:
> +python __anonymous () {
> +    systemd_packages = "${PN}"
> +    pkgconfig = d.getVar('PACKAGECONFIG', True)
> +    if ('openvpn' or 'vpnc' or 'l2tp' or 'pptp') in pkgconfig.split():
> +        systemd_packages += " ${PN}-vpn"
> +    d.setVar('SYSTEMD_PACKAGES', systemd_packages)
> +}

I'm not keen on this as its quite a lot of logic to put in the recipe
when it would be good to handle this case of optional packages in the
systemd class itself.

However I'm not sure how best to do this as the FILES_* manipulation
needs to happen *before* the package split, and this check can only
happen *after* the package split.  I wonder if it's possible to split
up the systemd class functionality like that.

So, we should probably merge this now, and try and fix the class so we
can remove it again later.

Ross



More information about the Openembedded-core mailing list