Re: PATCH: fix for: bug with ampersand modules, cvs client, and tags

[Greg Klanderman]

Hi Derek,

Thanks!  Very glad to have that fixed.  I'm guessing the second bug
you refer to below is new, as we're not having any promlem there with
cvs 1.11.  I'll take a closer look at sanity.sh.

thanks
Greg

>>>>> Derek R Price <dprice@collab.net> writes:

> I managed to reproduce this in the dev version in local mode.  This patch 
> _does_
> fix the local behavior.  Unfortunately I discovered another bug which seems 
> to be
> preventing the checkout of ampersand modules (maybe just the second module and
> maybe just with '-d') within another module entirely.  i.e.:

>     mod    dirname &ampermod

> fails to check out ampermod entirely.

> In any case, I have checked in Greg's fix and I will document the second bug 
> with
> a test case shortly.

> Greg, for the future, it makes things much easier on us if you include a 
> sanity.sh
> test case which reproduces your bug conditions with your patch.  Failing that 
> a
> simple shell script which reproduces the bug without external data files is 
> second
> best.  Either should pass after the patch is applied.

> Derek
> --
> Derek Price                      CVS Solutions Architect ( http://CVSHome.org 
> )
> mailto:dprice@collab.net         CollabNet ( http://collab.net )
> --
> I will not torment the emotionally frail.
> I will not torment the emotionally frail.
> I will not torment the emotionally frail...

>           - Bart Simpson on chalkboard, _The Simpsons_

> Greg Klanderman wrote:

>> I've fixed the bug I reported below yesterday.  The ChangeLog entry
>> and patch against cvs-1.11 are attached.  Please let me know if you
>> need anything more.