Bug#709842: [PATCH] Add new helper dh_installsystemd

2013-06-10 Thread Joey Hess
Michael Stapelberg wrote: > AFAICT, there are no such binaries shipped with debhelper itself. Does > that imply that we should ship such a tool in e.g. systemd-dev built > from the systemd source package? Is that what you’re suggesting? You could put the tool into dpkg-dev, or something. I suppos

Bug#709842: [PATCH] Add new helper dh_installsystemd

2013-06-10 Thread Michael Stapelberg
Hi Joey, Thanks again for your constructive feedback. Joey Hess writes: > That's not what I was trying to say. I like debhelper commands to be > thin wrappers around programs that handle the heavy lifting and the > domain-specific knowledge. This is why debhelper has a per-command LOC > limit. >

Bug#709842: [PATCH] Add new helper dh_installsystemd

2013-06-10 Thread Joey Hess
Michael Stapelberg wrote: > Another important reason is that packages might only ship a service file > without actually shipping a sysvinit script. This affects not so much > actual daemons which are mandated by policy to ship a sysvinit script, > but for example dbus-activated services. I don't s

Bug#709842: [PATCH] Add new helper dh_installsystemd

2013-06-10 Thread Michael Stapelberg
Hi Joey, Thank you for your feedback. Comments inline: Joey Hess writes: > It seems that despite the "install" in its name, dh_installsystemd does > not install anything. It does some registration of systemd services > files already installed by the package's Makefile or by dh_installinit. > So,

Bug#709842: [PATCH] Add new helper dh_installsystemd

2013-06-10 Thread Joey Hess
Michael Stapelberg wrote: > The raison d'être for dh_installsystemd is the fact that some packages > ship multiple service files but only a single SysV init script, i.e. > there is no 1:1 correlation between systemd unit files and SysV init > scripts. As an example, samba consists of smbd and nmbd,

Bug#709842: [PATCH] Add new helper dh_installsystemd

2013-06-09 Thread Michael Stapelberg
Hi Joey, friendly ping? Is there something that needs to be changed/clarified before you can merge this patch? I saw that you have committed other patches since I sent this bugreport :). Thanks. -- Best regards, Michael -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org wi

Bug#709842: [PATCH] Add new helper dh_installsystemd

2013-05-25 Thread Michael Stapelberg
Package: debhelper Version: 9.20130518 Severity: wishlist Tags: patch X-Debbugs-CC: pkg-systemd-maintain...@lists.alioth.debian.org Hi Joey, I have been working on dh_installsystemd lately and I am confident that it is ready for inclusion into debhelper after being tested for several weeks by sev