[PATCH 4/4] Mark nscd service as forking in systemd service file (#16639)

[Rich Felker]

On Wed, Feb 26, 2014 at 10:52:42PM +0530, Siddhesh Poyarekar wrote:
> Hi,
> 
> Currently the nscd service is installed in systemd as a simple
> service, which means that it is able to handle its own errors and does
> not quit.  Since nscd does not fit that description, i.e. it can exit
> on errors like, say, failing to parse nscd.conf, it should be declared
> as forking instead.  Tested on Fedora.

I don't think this is the right meaning of "forking". Forking just
means the daemon backgrounds itself so that its parent cannot manage
its lifetime with a pid. If this behavior cannot currently be fixed in
nscd via a command line option, it should be fixed at the source
level. Forking is always wrong for daemons.

>  [Service]
> -Type=simple
> -ExecStart=/usr/sbin/nscd --foreground
> +Type=forking
> +ExecStart=/usr/sbin/nscd

It looks like the --foreground option is correct and this patch breaks
something that was working before.

>  ExecStop=/usr/sbin/nscd --shutdown
>  ExecReload=/usr/sbin/nscd -i passwd
>  ExecReload=/usr/sbin/nscd -i group

These look incorrect though.

Rich