[merge] updated store-escaping for windows

Martin Pool mbp at sourcefrog.net
Wed Apr 12 07:35:46 BST 2006


On 12/04/2006, at 2:39 PM, Robert Collins wrote:

> On Tue, 2006-04-11 at 18:57 +1000, Martin Pool wrote:
>> This change, based on John's previous work, changes Knit stores to  
>> avoid
>> characters in filenames that can't be safely stored on Windows.
>> (Basically everything but lowercase letters, digits, and a few
>> punctuation characters.)
>>
>> - Add a FakeVFAT transport decorator that lets us check some of these
>>   constraints.  (Eventually it might be nice to be able to run
>>   WorkingTree tests through this too to trap Windows-specific bugs
>>   even when testing on Unix.)
>>
>> - Add an 'escaped' parameter to the TransportStore.
>
> Some feedback
>
> the propogation of _cloning in some function calls is undesirable -  
> that
> was a hack to allow the legacy formats - the non metadir ones - to
> privately signal that they did not need to create their data  
> structures.
> Certainly it shouldn't be part of the public api.

OK, this additional change confines it to the classes which need it:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 20060412-cleanup-cloning.diff
Type: application/octet-stream
Size: 4118 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060412/63e1034c/attachment.obj 
-------------- next part --------------

-- 
Martin





More information about the bazaar mailing list