Andrey Aristarkhov wrote:

>Hi!
>
>I've found a bug in loginfo processing. The problem is that if file to
>be committed contains spaces, for instance, "Read Me.txt", format string
>%{sVv} expanded like 
>(1)    Read Me.txt,oldRev,newRev, 
>but should be 
>(1)    Read\ Me.txt,oldRev,newRev
>In first case filter script takes 2 paramters and in second case -
>single parameter.
>
>I've wrote a patch to solve this problem. Please apply in ccvs/src
>directory.
>
>Regards,
>Andrey Aristarkhov
>BiTechnology 
>  
>

The problem with that is that it would break backwards compatibility, 
though it is possible that noone is using file names with spaces in them 
since this has never worked.  I don't believe it is documented this way 
either.

I don't suppose you would be willing to test out this patch: 
<http://ximbiot.com/ccvs.newfmtstrings.1-11.diff>?  Among other things 
it unifies the format of the *info file command templates and IIRC 
provides for paths with spaces by quoting them or the like while 
allowing for backwards compatibility.

I've been trying to get this patch tested for awhile now.  A few people 
have volunteered but never gave any feedback.  Hopefully it still 
applies without trouble.  There should be docs and test cases included. 
 I believe the patch was against 1.11.

Let me know if there's anything else I can do.

Derek

-- 
                *8^)

Email: [EMAIL PROTECTED]

Get CVS support at http://ximbiot.com
-- 
I will not drive the principal's car.
I will not drive the principal's car.
I will not drive the principal's car...

          - Bart Simpson on chalkboard, _The Simpsons_





_______________________________________________
Bug-cvs mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/bug-cvs

Reply via email to