merge 2531 3410
thanks

Nathanael Nerode writes ("Re: PATCH: Make install-info use fcntl (via perl's 
flock) for locking"):
> unmerge 3410
> thanks
> # 3410 is a different bug, requesting that the lockfile be
> # cleaned up on process kill

The only correct fix to that is to use fcntl instead, so it should
remain merged.

> Hmm.  In that case, what is the *point* of using fcntl locking?  *EVER*?
>   Fcntl locking is a fscking waste of time and effort in this case.
> 
> Is it simply to make sure that the lock file is deleted if the program
> dies?

Yes (well, there are other advantages too but that's the main one).

>   This can be done in a much simpler way, without a lock-method
> transition, by catching kill signals and deleting the lock file on all
> exit paths.  As suggested in bug 3410, which was mistakenly merged with
> this one.

This is not reliable.

I'm afraid I don't have time right now to explain all of this to you
but all of the stuff I've been explaining should be well-known to
anyone messing around in this area.

Regards,
Ian.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to