[cvsnt] Re: Binary files are imported as text files (CVSNT 2.5.02 #2096)

[Thomas Singer]

> You're sending:
> 
> import -m initial import -ko -I! -W *.bin -k 'b' 

As you know, SmartCVS doesn't use command line CVS, but send the requests 
directly.

> You are forcing -ko in your upload - this overrides any wrappers (forcing stuff in import is generally a bad idea).

It worked in CVSNT 2.0.58d and still works in any half-decent GNU CVS 
version, so I guess it is a newly introduced incompatibility in CVSNT 2.5.*.

> import -m initial import -k-v -I! -W *.bin -kb 

Are you sure this will also work with GNU CVS?

> import -I! -W *.* -ko -W *.bin -kb

Does this also works for files without a period?

> Or the best way in this case - don't specify anything and let the server handle it (*.bin is already default binary).

I doubt, that this is the best (BTW, "bin" was just a sample extension, it 
also could have been "foo", "bar" or "bazz"). SmartCVS (like WinCVS?) scans 
the files for "mysterious" characters and then suggests, whether it is a 
binary or text files, but the user of course can change it manually.

--
Best regards,
Thomas Singer
_____________
smartcvs.com
smartsvn.com


Tony Hoyle schrieb:
> Thomas Singer wrote:
> 
>> Hi Tony,
>>
>> While running our SmartCVS regression tests I've found out that CVSNT 
>> 2.5.02 #2096 imports even binary files with the text file keyword 
>> substitution mode. This used to work in CVSNT 2.0.58d and still works 
>> in GNU CVS 1.11.14, our "reference" server.
> 
> 
> You're sending:
> 
> import -m initial import -ko -I! -W *.bin -k 'b'
> 
> You are forcing -ko in your upload - this overrides any wrappers 
> (forcing stuff in import is generally a bad idea).
> 
> There are at least two ways to achieve what it looks like you're trying 
> to do:
> 
> import -m initial import -k-v -I! -W *.bin -kb
> 
> or
> 
> import -I! -W *.* -ko -W *.bin -kb
> 
> Or the best way in this case - don't specify anything and let the server 
> handle it (*.bin is already default binary).
> 
> Tony
> _______________________________________________
> cvsnt mailing list
> cvsnt at cvsnt.org
> http://www.cvsnt.org/cgi-bin/mailman/listinfo/cvsnt
>