restart a script in etc/rc.d

[Doug Barton]

Oliver Lehmann wrote:
> Doug Barton wrote:
> 
>> Artis Caune wrote:
>>> There is logic error in bacula rc.d script. It should first set
>>> default variables and only then use them.
>>>     pidfile="${bacula_fd_pidfile}"
>>>     : ${bacula_fd_pidfile="/var/run/bacula-fd.9102.pid"}
>>>
>>> If you don't set pidfile in rc.conf, pidfile is "" so it kills all bacula-fd's
>> Yes, I tried to convince the OP to fix this, but he thought he had it
>> covered, so I'm giving him the opportunity to prove me wrong. :)
> 
> Yeah this did it - Its working now.
> I thought you where just telling me that to have another pidfile for the
> 2nd start script - I missed the point that it was empty for the first one
> and because of that it is falling back to ps...
> 
> I'll probably write a PR and commit it after aproval for PROVIDE and the
> pidfile setting...

Excellent. :)


Doug

-- 

    This .signature sanitized for your protection