Hi Joey, hi Gergely, On 24.10.2012 16:18, Joey Hess wrote: > Gergely Nagy wrote: >> Another option could perhaps be to ship the files in /lib, but instead >> of relying on UCF or some other kind of mechanism to fiddle with the >> symlinks in /etc, ship files there too. Files that do nothing but >> .include the file in /lib. >> >> Things like: >> >> .include /lib/systemd/system/foo.service >> >> The advantage of this is that the whole conffile handling is left up to >> dpkg, no other mechanism is needed, neither in maintainer scripts, nor >> elsewhere. Users can still modify the files, or even remove them: it >> will be handled correctly. In most cases, they can even leave the >> .include there, and just override the few things they need to override. >> >> (and dh_installinit could then perhaps assist with creating these >> .include-only files automatically, based on what the package installs >> under /lib/systemd) > > Yes, it certianly could. Choice is entirely up to the systemd maintainers..
Tollef, Michael S. and I discussed this. We do like the .include mechanism and prefer it over copying the whole .service file. That said, shipping such an .include-only file in /etc/ by default for every package providing a .service file will clutter /etc notably and we are worried about that, especially since overriding settings from .service files will be the exception rather then the norm. So we don't think dh_installinit should create such .include-only files (at least not by default). For now we will try to address this issue with better documentation e.g. in README.Debian. Another idea could be having a small helper tool, like say "systemd-edit $foo.service", which could assist in creating such override/.include files. This is something we can work on in the systemd package though, so I don't think we should bother Joey with this. I'd be fine if the patch is merged as-is. Cheers, Michael -- 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