Bug#759001: [systemd-devel] sysv-generator: doesn't handle /etc/insserv/overrides or /etc/chkconfig.d

[Christian Seiler]

Control: tags -1 + patch

(remove CC systemd-devel, add CC corresponding Debian bug)

Am 16.02.2015 um 19:12 schrieb Michael Biebl:
> That said, as one of the Debian systemd maintainers, I could probably
> be convinced to add the insserv override support as a downstream patch
> for jessie.

I've attached a minimal patch for Debian Jessie for this. I tested it
briefly, i.e. if no /etc/insserv/overrides/$NAME exists, nothing
changes, but if it exists, it properly overrides the current LSB
script. Please review.

Only minor issue is that SourcePath= is a single-valued entry in
systemd, so I can't just add the override if it exists, and
FragmentPath= is only ever dynamically filled by systemd when parsing,
so I can't set that either. It's mostly cosmetic, but systemctl status
for example won't show that the file was overwritten.

(Alternatively, one could ONLY use /etc/insserv/overrides/$NAME as
SourcePath=, of course, I don't have a strong opinion on either one of
these solutions.)

> We are pretty late into the freeze though, so this would
> require an ack from our release managers.

Do you want me to ask for pre-approval?

Christian

-------------- next part --------------
A non-text attachment was scrubbed...
Name: sysv-generator-add-support-for-etc-insserv-overrides.patch
Type: text/x-patch
Size: 6233 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-systemd-maintainers/attachments/20150217/ff714aa6/attachment-0001.bin>