Re: [PATCH v2] cups: update systemd support

Burton, Ross
2015-12-11T21:39:40Z

[-- Attachment #1: Type: text/plain, Size: 1406 bytes --]

On 22 October 2015 at 15:00, Andrew Shadura <andrew.shadura@collabora.co.uk>
wrote:

> Add PACKAGECONFIG for systemd, enable it when systemd is enabled
> in DISTRO_FEATURES. Pass the correct path to systemd units,
> ensure they're installed into the package.
>
> Drop custom systemd unit files, use those shipped by the upstream
> instead.
>

Sorry for the delay in looking at this. buildhistory-diff reports this:

packages/corei7-64-poky-linux/cups/cups: FILELIST: removed
"/etc/rc2.d/S81cups /lib/systemd/system/cups.service /etc/rc5.d/S81cups
/etc/rc5.d/K36cups /etc/rc0.d/K36cups /etc/rc2.d/K36cups /etc/rc3.d/S81cups
/etc/rc3.d/K36cups /lib/systemd/system/cups.socket
/lib/systemd/system/cups.path /etc/init.d/cups" added
"/lib/systemd/system/org.cups.cupsd.socket
/lib/systemd/system/org.cups.cupsd.service
/lib/systemd/system/org.cups.cups-lpd@.service
/lib/systemd/system/org.cups.cupsd.path
/lib/systemd/system/org.cups.cups-lpd.socket"

Looks like if systemd is enabled it removes the sysvinit files, but we
still need those installed if sysvinit is in DISTRO_FEATURES.

Also the configure script checks for libsystemd-daemon which we don't build
by default, so we'll either need to enable compat in systemd or patch
configure to look for the new library name. To be honest so much is
silently breaking that I think we should enable compat in systemd...

Ross

[-- Attachment #2: Type: text/html, Size: 2110 bytes --]