On Sat, Jul 13, 2013 at 04:12:33PM +0200, Zbigniew Jędrzejewski-Szmek wrote:
> On Mon, Jun 17, 2013 at 07:11:50PM +0100, Ross Lagerwall wrote:
> > Before, "systemctl reenable [email protected]" would fail with:
> > Failed to issue method call: File exists
> > To fix this, reimplement "reenable" explicitly as a disable followed by
> > an enable.
> > This is shorter and is how the man page documents its behavior.
> > ---
> >  src/shared/install.c |   38 +++++---------------------------------
> >  1 file changed, 5 insertions(+), 33 deletions(-)
> Hm, I don't get this error with "reenable", but your patch indeed 
> simplifies things, so I don't see a reason not to apply it: applied now.


As far as I can recall, it would fail if
/etc/systemd/system/getty.target.wants/[email protected] is set up as a
symlink to /usr/lib/systemd/system/[email protected] and then "systemctl
reenable [email protected]" is run.  I can't exactly remember but I
think it would fail because "reenable" would try to delete the
untemplated unit "[email protected]" (which would not exist) but then try
to create the templated unit "[email protected]" which would still
exist hence the "File exists" error.  Thanks for applying the patch
anyway.

-- 
Ross Lagerwall
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to