[cvsnt] commit of -kB file gives error message in cvsdelta.cpp

[David Buntin]

Arthur,

I have been searching the archives and found this thread. We are
experiencing a similar problem. We have a number of files that report the
following error when we attempt to checkout an old revision:

cvs update -P -d -r 1.1 -- big_binary_test.xls (in directory
C:\dev\Test_Module_B)
cvs [server aborted]: received abort signal
cvs: cvsdelta.cpp:558: bool cvsdelta::patch(const ByteArray&, const
ByteArray&, ByteArray&): Assertion `pos+len<=file1.size()' failed.

You mention an update (2.5.04) with a cvsdelta fix is coming soon. Do you
know if it will fix this problem too?

Thanks,
David Buntin


>Emma,
>
>> The comments for the two changes to cvsdelta.cpp are:
>> rev 1.5.2.22 Make BOM invisible to diff as it just confuses people.
>> rev 1.5.2.23 cvsdelta corner case
>> 
>> We took that change, built it into our cvsnt and tested it 
>> with the file 
>> that showed the error. It appears to have resolved the problem.
>> 
>
>Those changes are to 2.5.04 - I suggest you try upgrading to the latest
>2.5.04 testing release on a 'test' box and validate you can upgrade to
>that as soon as it goes stable (very soon - within the next couple of
>weeks probably).
>
>Regards,
>
>
>Arthur