[Pkg-Cyrus-imapd-Debian-devel] Re: Cyrus 2.2 packages in Debian Sarge

[Sven Mueller]

Henrique de Moraes Holschuh wrote on 24/03/2005 03:28:
> On Wed, 23 Mar 2005, Sven Mueller wrote:
> 
>>Another thing just crossed my mind when speaking of help needed: I 
>>transplanted all the patches that made sense to me from the cyrus21 to 
>>the cyrus22 package. This includes a patch which adds timeouts to 
>>lib/lock_*.c and I'm not 100% sure they do the right thing. Is anyone on 
>>this list capable of checking that? I know that the patch doesn't break 
>>Cyrus, but I'm not sure it is effective.
> 
> For what is it worth: I would NOT run production without these patches, in
> 2.1.  Unless the code in 2.2 changed a LOT, you really want those patches.

I'm with you on this. I have seen while applying the patches that the 
2.2 code does check for received signals (including SIGALRM), but it 
doesn't look as if it is setting an alarm timer when creating locks. 
That's why the patch is only partially applied: The signal handling was 
mostly in place already. I'm just not 100% sure the applied patch part 
does the trick.

cu,
sven