Bug#627339: init script should use QUIT instead of TERM

2011-05-20 Thread Ondřej Surý
Hi Steven, looks good, but you can use --retry=0/30/KILL/5 option for retrying. I'll use your patch as a base and rewrite the init.d script with regards to current /etc/init.d/skeleton... O. On Thu, May 19, 2011 at 19:25, Steven Kurylo steven.kur...@aviawest.com wrote: Package:

Bug#627339: init script should use QUIT instead of TERM

2011-05-20 Thread Ondřej Surý
Hi Steven, could you try attached init.d script? I rewrote a big part of original init.d script using start-stop-daemon and lsb init functions as much as possible, so something might be broken now. O. -- Ondřej Surý ond...@sury.org cyrus-common.cyrus-imapd.init Description: Binary data

Bug#627339: init script should use QUIT instead of TERM

2011-05-20 Thread Steven Kurylo
On 11-05-20 06:45 AM, Ondřej Surý wrote: Hi Steven, could you try attached init.d script? I made some minor corrections. * Silenced pidofproc in do_start and try-restart * Fix pidofproc return code handling in do_stop (otherwise /etc/init.d/cyrus-imapd stop always returned failed) I'll

Bug#627339: init script should use QUIT instead of TERM

2011-05-20 Thread Ondřej Surý
On 20.5.2011, at 19:00, Steven Kurylo steven.kur...@aviawest.com wrote: On 11-05-20 06:45 AM, Ondřej Surý wrote: Hi Steven, could you try attached init.d script? I made some minor corrections. * Silenced pidofproc in do_start and try-restart * Fix pidofproc return code handling in

Bug#627339: init script should use QUIT instead of TERM

2011-05-19 Thread Steven Kurylo
Package: cyrus-common-2.4 Version: 2.4.8-7 install-configure.html says we should use QUIT first: Since a clean shutdown may never finish if a child process is stuck for some reason the recommended approach is to send a SIGQUIT then loop on the master process sending a signal 0 every second