[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: advisory locks patch

From: Mark D. Baushke
Subject: Re: advisory locks patch
Date: Tue, 31 Aug 2004 02:33:27 -0700

Hash: SHA1

Matthew Ogilvie <address@hidden> writes:
>     I think I've fixed the C89 problems (see latest minor update), but
> don't have an easily accessable older compiler to try it on.

You have indeed fixed the C89 problems.

With regard to my previous comments, I do understand that 'cvs edit'
presently only deals with locking a file regardless of version and that
'cvs edit -c' will do the same. 

I still believe it would be desirable to be able to lock a particular
branch of the file as a future addition to advisory locks in general.
I do not mean to imply that this enhancement needs to provide that
extension at this time.

I still believe that administrators of a cvs repository might find it
desirable that the -c behavior were the default for any 'cvs edit' and I
therefore suggest that it may be desirable to introduce a new
CVSROOT/config option to make that the default. 

The current patch under consideration which is found in attachment

for this issue:


applies with a slight amount of fuzz to the cvshome.org tree does
compile with a C89-based compiler. I have not tried running the
sanity.sh test as yet.

As has been mentioned, the patch is very large, but I think it does
introduce a missing functionality that could be useful to a fair number
of folks out there and I do not object to this patch being added.

        -- Mark
Version: GnuPG v1.2.3 (FreeBSD)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]