Hi Michael, On 05.02.2013 15:25, Michael Meskes wrote: >> Michael, if you want me to review the relevant changes or provide a >> patch, just let me know. > > A patch would be great. Thanks.
The patch would basically be as simple as: diff --git a/debian/acpid.service b/debian/acpid.service index 733edc9..b998711 100644 --- a/debian/acpid.service +++ b/debian/acpid.service @@ -5,3 +5,6 @@ Requires=acpid.socket [Service] StandardInput=socket ExecStart=/usr/sbin/acpid + +[Install] +WantedBy=multi-user.target This way acpid is also started via multi-user.target, which basically is runlevel 2-5 in Debian. I could have added "Also=acpid.socket" to the [Install] section. What that does is, that a "systemctl enable acpid.service" also enables the corresponding socket unit (and vice versa for disable) In case of acpid it might make sense that the service and socket unit can be enabled and disabled independently, e.g. for people which only need acpid to be running on-demand. If you don't think there is a use-case for that, you can decide to add the Also=acpid.socket line. In case you were wondering why I didn't update debian/acpid.links: The reason why this is not necessary, are the changes in systemd 44-9 [1] which will automatically enable systemd units and sockets. So if you want, you can actually drop debian/acpid.links again. Cheers, Michael [1] http://anonscm.debian.org/gitweb/?p=pkg-systemd/systemd.git;a=commitdiff;h=d7e578179529 -- Why is it that all of the instruments seeking intelligent life in the universe are pointed away from Earth?
signature.asc
Description: OpenPGP digital signature
