[MERGE] Container format version 2

Robert Collins robertc at robertcollins.net
Tue Jul 17 11:51:26 BST 2007

On Tue, 2007-07-17 at 15:44 +1000, Andrew Bennetts wrote:
> This primarily updates the container format to allow whitespace in
> record names.
> The test suite uses file IDs with whitespace (e.g.
> TestFileIdInvolvedSuperset in
> bzrlib/tests/repository_implementations/test_fileid_involved.py), and
> even if
> the intent was to disallow it, bzrlib allows it and has for some time,
> so it's
> probably not safe to assume there are no file IDs with whitespace in
> the wild.
> So in this bundle the only disallowed byte in a record name is a
> '\n'. 

And presumably \x00 ?

My index code is as strict as container 1 formats. I'd like to do a scan
of all the repositories on the supermirror before we loosen this because
I strongly suspect we don't have space containing revids out there, and
don't see a good reason to have them in the core- it limits our
fast-parsing options more than a little.

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/20070717/7d4e2e73/attachment.pgp 

More information about the bazaar mailing list