bug-cvs
[Top][All Lists]
Advanced

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

Re: do_verify patch for cvs top-of-tree


From: Derek R. Price
Subject: Re: do_verify patch for cvs top-of-tree
Date: Wed, 22 Aug 2001 14:20:50 -0400

"Mark D. Baushke" wrote:

> See the patch attached to http://ccvs.cvshome.org/issues/show_bug.cgi?id=19
>
> I added some more documentation for the new option and enhanced the
> example verification script to do something to modify the log message.
>
> I didn't go into the reasons that a user may want to choose 'always'
> vs 'stat' as I thought I should have you double-check my understanding
> of how often the verifymsg script is being run. Let me know if you
> think there needs to be another paragraph on that issue or not.

You were right - I was thinking that a log message would only need to be
verified once per editor session, but a verifymsg script could be accessing
`pwd` and if it is then it needs to be called in each dir.  I'll remove the
stat stuff from the code and docs when I check in your patch unless I find
some major issue to pass back to you.


> > Any differeing opinions?  The history of the issue is here:
> > http://ccvs.cvshome.org/issues/show_bug.cgi?id=19 and I attached
> > Mark's new patch to it rather than forward it.
>
> I did the same.

I'll review the rest of the patch more thoroughly soon and annotate the issue
or commit it.

Derek

--
Derek Price                      CVS Solutions Architect ( http://CVSHome.org )
mailto:dprice@collab.net         CollabNet ( http://collab.net )
--
If there be any among us who would wish to dissolve this Union or to change its
republican form, let them stand undisturbed as monuments of the safety with
which error of opinion may be tolerated where reason is left free to combat it.

                        - Thomas Jefferson; 1st Inaugural, 1801






reply via email to

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