Re: [PATCH] CVSREADONLY feature

[Mark D. Baushke]

Larry Jones <lawrence.jones@eds.com> writes:

> Mark D. Baushke writes:
> >  
> > > 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."
> > 
> > I issued that message with write mode. I am not sure it should be done
> > for a read-only operation or when the -Q switch is used. Obviously, I
> > can extend it further if you feel it is needed.
> 
> I think Derek's point was that you should *always* issue that message;
> this "feature" is extremely dangerous and should not be used lightly. 
> (I've never liked it, but I recognize the need it fills and I haven't
> been able to come up with a better alternative.)  I'd say you should
> issue the message as soon as the option is parsed, although I'm open to
> suppressing it for quiet and/or really quiet mode.

Okay, I have applied such a patch. I supress the warning for quiet and
really quiet mode. The warning is issued after all the options are
parsed to allow for -Q -R and -R -Q option ordering.

        -- Mark