[Pkg-clamav-devel] Redo of the initscripts

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Sun Apr 13 15:48:21 UTC 2014


Hi Sebastian,

On 09.04.2014 12:56, Sebastian Andrzej Siewior wrote:
> I think I completed the re-do of the init scripts for freshclam, clamd
> and milter. I did my last tests yesterday.
> The primary motivation was to close #580188. The use a PID file is now
> required. start-stop-daemon uses it now to kill the proper daemon. I
> also use "pkill -0" to check if the daemon is running.
> The change in clamav where I add the extra \n to the pidfile is in
> upstream's bugilla as #10907 and should be part of 0.98.2.

Thanks for modernizing the init scripts. (The old method of stopping the 
service was really ugly.)

> The whole thing is in the bigeasy_init branch. Could someone please take
> a look at it before it gets into unstable?

I reviewed the new init files and they look good. I just removed some 
now unused code.

One thing I noticed is that your patch is not integrated via git-dpm as 
the other patches are and also duplicates the number 0001.
I think the best way to fix this is to revert your commit adding the 
patch, then use 'git-dpm checkout-patched', commit your fixes to the 
upstream sources and finally use 'git-dpm update-patches'.

Otherwise this can be merged to unstable.

Best regards,
Andreas



More information about the Pkg-clamav-devel mailing list