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

Ondřej Surý ondrej at sury.org
Fri May 20 17:52:13 UTC 2011


On 20.5.2011, at 19:00, Steven Kurylo <steven.kurylo at 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 do_stop (otherwise /etc/init.d/cyrus-imapd stop always returned failed)

Yeah, I found this when testing the packages, but was too lazy to send you updated patch.

> I'll have to so more testing of the sync_client.  One thought is there is may be race condition with cyrmaster (or a user) starting a new sync_client process after sync_stop has run, but before the shutdown is finished.
> 
> Perhaps the touch "$SYNCSHUTDOWN" should be done regardless and rm -f $SYNCSHUTDOWN should be done after cyrmaster has stopped.  I would assume if "$SYNCSHUTDOWN" exists, then no new sync_client will start.

Lemme know how it goes. I needed to fix mipsel build anyway, therefore I did the upload right now.

O.
> 





More information about the Pkg-Cyrus-imapd-Debian-devel mailing list