bug-cvs
[Top][All Lists]
Advanced

[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

-----BEGIN PGP SIGNED MESSAGE-----
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
http://sourceforge.net/tracker/download.php?group_id=20011&atid=320011&file_id=99545&aid=822306

for this issue:

http://sourceforge.net/tracker/index.php?func=detail&aid=822306&group_id=20011&atid=320011

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
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (FreeBSD)

iD8DBQFBNEXn3x41pRYZE/gRAoNPAJ9YnxdVRDmEdlAcpm+FiUVDdGW9uACeNYUJ
cQLRvJXEo+e3wkDHiy14aFQ=
=lFtv
-----END PGP SIGNATURE-----




reply via email to

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