|
From: | Derek Robert Price |
Subject: | Re: [PATCH] CVSREADONLY feature |
Date: | Thu, 06 Mar 2003 14:29:14 -0500 |
User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.2) Gecko/20030208 Netscape/7.02 |
Mark D. Baushke wrote:
if (current_parsed_root != NULL && current_parsed_root->isremote) { + if (readonlyfs) + error (1, 0, "Read-only repository feature unavailable"); + /* Create a new list for directory names that we've sent to the server. */ if (dirs_sent_to_server != NULL) dellist (&dirs_sent_to_server); dirs_sent_to_server = getlist (); }
I don't see why not. It would at least, hopefully, discourage people from putting -R in their .cvsrc, a move that could cause all sorts of trouble.
I'd put the sanity check and error message at the end of the parse_cvsroot() function in src/root.c, though, with the other CVSROOT sanity checks. And make the error message more complete. Something like "Read-only repository feature unavilable with remote roots (cvsroot = %s)."
I'd also change the write lock error message to make it more clear to a novice that the write lock failed due to an option they passed in on the command line and not some feature of the repository they are accessing. Instead of, "write lock failed - read-only repository", how about something more like, "write lock failed due to read-only CVS option (cvs -R)".
And one more, how about always issuing a warning? Something like, "WARNING: Read-only repository access mode selected via `cvs -R'. Using this option to access a repository which some users write to may cause intermittant sandbox corruption."
Derek -- *8^) Email: derek@ximbiot.com Get CVS support at <http://ximbiot.com>! -- Life in a vacuum sucks.
[Prev in Thread] | Current Thread | [Next in Thread] |