On 24 Jun 2016 5:00 a.m., "intrigeri" <intrig...@debian.org> wrote:
>
> Hi,
>
> Felipe Sateler wrote (06 Jun 2016 23:49:46 GMT) :
> > Please find attached a unit that wraps the currently existing init
> > script.
>
> Thanks! I've integrated something along these lines and expect I'll
> upload today.

Cool, thanks!

> > Also, apparmor init script is not stopped on shutdown (and thus I did
> > not add a Conflicts on shutdown.target), you might want to consider
> > dropping the ExecStop in that case.
>
> What would it buy us to drop ExecStop? Even though the service is not
> stopped on shutdown (that's on purpose), it may be useful to support
> manual "systemctl stop apparmor.service".

Mostly to prevent user errors. If the stop functionality is useful for
debugging purposes, maybe it is safer to ship in a helper script.

Saludos

Reply via email to