On Sun, Jul 01, 2012 at 11:17, Tobias Stoeckmann wrote:
> Your patch breaks this compatibility for every branch except the initial
> one.  GNU CVS is not consistent about its behavior in many ways.
> 
>> Ha! This is the same bug I was looking into last week.  I wasn't sure in
>> what circumstances adding the magic (like wtf?) is necessary.  I'd say
>> this is probably ok.
> 
> I disagree:
> 
> $ pwd
> /home/tobias/src/lib/libssl/src/crypto
> $ cvs log mem.c
> RCS file: /cvs/src/lib/libssl/src/crypto/mem.c,v
> Working file: mem.c

> 
> 
> Magic, magic everywhere!  Except in this special initial branch one. ;)
> 
> If there is no need to be compatible to GNU CVS as strictly as the project
> initially tried to, I'm fine with that.  But as far as I know, this patch
> might break scripts and other tools that expect this weird behavior.

I had trouble finding a decent file where there was a difference,
thanks.  I guess we need an uglier patch to only skip magic if the
version is 1.1.1?

Reply via email to