Am 28.01.2015 um 15:52 schrieb Mathieu Parent: > 2015-01-28 15:42 GMT+01:00 Michael Biebl <bi...@debian.org>:
>> Do you use dh-systemd/init-system-helpers to register your cdtb.service >> unit? It generates maintainer scripts code to ensure "systemctl >> daemon-reload" is called at the correct times. > > I use dh_systemd with overrides: > http://anonscm.debian.org/cgit/pkg-samba/ctdb.git/tree/debian/rules > > ctdb.postinst (in the .deb) ends up with only: > > if [ -x "/etc/init.d/ctdb" ]; then > update-rc.d ctdb defaults >/dev/null || exit $? > fi > > and /etc/init.d/ctdb is indeed executable (Is it on your machine Martin?). > > It seems that update-rc.d calls dameon-reload in some circumstances. > Or should ctdb.postinst call it directly? Two issues: 1/ update-rc.d should call daemon-reload for sysv init scripts, it currently doesn't 2/ in your case you ship a native service file, *but* you you install the service file in override_dh_installinit. dh_systemd_enable runs *before* dh_installinit, so the helper doesn't pick up the file and doesn't generate any code for it. My suggestion would be, that you either convince upstream to ship that file upstream (and have it installed by make install) or install it via debian/ctdb.install. dh_install runs before the dh_systemd_* If you fix the installation of the ctdb.service file, the dh-systemd helper should generate maintainer scripts code for that service. Btw, I think you can drop the override_dh_systemd_enable override, it doesn't pass any relevant arguments to dh_systemd, so it seems superfluous. [1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=774799 -- 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
