On Fri, 11.04.14 09:48, Michael Olbrich (m.olbr...@pengutronix.de) wrote: > > > + else if (allow_restart && > > > > I would drop the "else" here, I think. Is there a reason not to do the > > restart thing anyway? If it is configured, it should run I think, just > > in case the failure action doesn't work or so... > > Are you sure? With Restart=always and FailureAction=reboot this would > try to start the unit while shutting down. Will the Conflicts= from the > default dependencies handle this correctly?
It should handle this correctly, and if it doesn't we should fix this. I mean, my thinking here is that combining FailureAction= and Restart=failure might not make much sense but there isn't really a strong reason to totally prohibit it... Lennart -- Lennart Poettering, Red Hat _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel
