Knits escape too many characters

Martin Pool mbp at sourcefrog.net
Sat Apr 22 00:01:56 BST 2006


On 22/04/2006, at 4:57 AM, Aaron Bentley wrote:

> It was mentioned in "[merge] updated store-escaping for windows",  
> and a
> few earlier emails.  I recall Martin saying "I've just realized  
> that we
> must escape uppercase letters on Windows", in an email, but I can't  
> find
> it quickly.

That's what I said - perhaps we should have had more discussion.

As you have probably gathered, the problem is that people could  
possible produce the ids "A" and "a" and commit them both, and that  
could cause havoc on non-case-sensitive filesystems.  The current  
format is just url-escaping, which is what was previously agreed,  
though the idea of doing it on uppercase characters was not discussed.

I'm open to changing it, though perhaps that can wait for a future  
format rev?

-- 
Martin







More information about the bazaar mailing list