Illegal Filesystem characters in revision names

[Robert Collins]

On Mon, 2005-12-05 at 22:07 +1100, Martin Pool wrote:

> > 3) Just switch to using a revision.weave file. Since revisions are no
> > longer saved directly by their name, we can break ourselves from being
> > limited to filesystem constraints. This would also require a branch
> > format bump.
> 
> Even if we do that, we still have to answer this question for file-ids.
> And I think even if we don't expect them to be stored that way, it will
> be very useful to have the option of mapping revision ids to files.
> 
> My preference would be to keep those characters out.  The only problem
> is people who've already done conversions from Arch.
> 
> > I'm hesitant to do this until we have knits. We already
> > have 1 file (inventory.weave) which scales by the number of commits, I
> > wouldn't really want to introduce a second one. (Though it would scale
> > better, since the number of lines per revision are much smaller).
> > 
> > Does anyone have more ideas? I'm leaning towards the third option, since
> > we kind of want to do it anyway. And I guess we can just say "arch
> > conversions aren't supported on windows until the next branch format".
> > Though it might be nice to get it sooner than that, since I've started
> > pointing my coworkers to bzr.
> 
> My preference would be for #1.

My preference is to escape characters we dont want, which could be
anything outside the legitimate URL character set (i.e. all non-7bit
ascii characters and some of those too). This makes it as simple as
possible for interoperation with other systems that have an analagous
concept. I.e. to convert CVS one might use the CVS repository prefix +
the dotted decimal revision to create a bzr rev id.

Rob

-- 
GPG key available at: <http://www.robertcollins.net/keys.txt>.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20051207/048bdf76/attachment.pgp