Hey Zbigniew,
Zbigniew Jędrzejewski-Szmek [2015-01-21 16:36 +0100]:
> On Wed, Jan 21, 2015 at 10:46:03AM +0100, Martin Pitt wrote:
> > Keeping track of which alias symlinks we actually want is error prone, and
> > restricting the creation of services for enabled init.d scripts would reduce
> > the utility of the generator (for manual starting disabled init.d scripts)
> > as
> > well as not cover the second case. So if we encounter an existing symlink,
> > just
> > remove it before writing a real unit.
> Looks fine. Although the code is clearer than the description :)
Heh. I removed the above rationale why it isn't done in a different
way, as it might be indeed overcomplicating the commit log.
> > + log_debug("loading SysV script %s", s->path);
> Capital "L"?
Fixed.
> Looks fine from my POV.
Thanks for the review! Pushed.
Martin
--
Martin Pitt | http://www.piware.de
Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel
