At 14:58 +0000 3/23/00, Tony Hoyle wrote:
>John Macdonald wrote:
>>
>> Tony Hoyle wrote :
>> || I thought of changing loginfo parameters so they are comma
>>separated... then you have the same problem, but with commas instead
>> || of spaces (although commas are far less common in filenames than spaces).
>>
>> A trailing ",v" on filenames is ubiquitous in CVS repositories. That
>> might not actually cause problems since CVS commands always deal with
>> the original name instead of the RCS name, but it is at least one
>> place where commas are far more common than spaces in filenames.
>>
>OK, but the users normally never see this.
>I stuck in a patch that escapes space (and, for good measure, '\')
>before it sends it to the scripts. That made it possible
>to write scripts that worked. However I'm a bit worried about the
>internal workings of CVS relying on there being no spaces.
>
>In the Win32 world spaces are very common, since you very rarely use
>the command line to type filenames. We have pre-existing
>projects with filenames in it that are almost complete sentences...
Same thing over here in the Macintosh world. MacCVS does not handle
this correctly from the GUI - only the AppleEvent interface seems to
work. Embedded <CR> in filenames seems to cause all sorts of obscure
problems at both ends (MacOS folder icons are called "Icon\r" and it
is sometimes nice to import them when they are part of a package like
PowerPlant).
And then there is the whole \r vs \n vs 0x0D vs 0x0A mess that we had
to sort out for MacCVS 3.1...In general, the cvs code is often rather
Unix-centric. Not surprising, but cross-platform filenames are
important in this day and age. Perhaps the URL convention of %xx
would be a good standard to use. Then again, I probably won't be
making such a change ;-)
- rmgw
http://www.electricfish.com/hawkfish/
----------------------------------------------------------------------------
Richard Wesley Electric Fish, Inc. [EMAIL PROTECTED]
"The professor is teaching me about 'tea'. It's very complicated."
- Leela in "Dr. Who: The Talons of Weng Chi-An"
