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