bug-cvs
[Top][All Lists]
Advanced

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

Re: preventing disturbance of branch tags.


From: Derek R. Price
Subject: Re: preventing disturbance of branch tags.
Date: Wed, 06 Jun 2001 15:53:24 -0400

Stephen Cameron wrote:

> Hi folks,
>
> I've updated my patch to "cvs tag/rtag" with sanity.sh test cases, ChangeLog
> entries, cvs.texinfo changes...  I think it's done, but more eyes looking it
> over couldn't hurt.
>
> It's here:
> http://www.geocities.com/dotslashstar/branch_patch.html#tagpatch
> Or the patch itself:
> http://www.geocities.com/dotslashstar/tagpatch.06.06.2001.txt

I checked this in.  Or rather, I checked in the version you had posted this
morning with a few bug fixes, new error messages, and new/fixed sanity test 
cases.

The bug fix I mention was adding a couple of "return (0);"'s where they were
missing and changing the operation of '-B' to strictly allowing the source tag 
to
be a branch (and forbidding it when missing).  This seemed simpler and more
consistent.

I did use the ChangeLog messages from the new patch, but a cursory inspection
showed that the only other differences were in cvs.texinfo, and I preferred the
first version of that with my changes.  The table you provided no longer applies
after my bug fix and you were going to leave me a lot of reformatting to do 
since
you neglected to use texinfo tags.

If you want to take a second shot at correcting our poor english and manual
organization, go ahead, but try and use the texinfo tags.  One hint is that if 
you
want to put something in quotes, you should probably be using a tag, and
CVS should be written @sc{cvs}.

Thanks for the patch!

Derek

--
Derek Price                      CVS Solutions Architect ( http://CVSHome.org )
mailto:dprice@collab.net         CollabNet ( http://collab.net )
--
"It's difficult to work in a group when you're omnipotent."

        -Q, "Deja Q"






reply via email to

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