[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] CVSREADONLY feature
From: |
Mark D. Baushke |
Subject: |
Re: [PATCH] CVSREADONLY feature |
Date: |
Mon, 17 Mar 2003 01:48:10 -0800 |
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