[MERGE] Stop-gap fix for Repository.get_revision_xml. (was Re: [MERGE] Don't use get_revision_xml when writing a bundle, instead get all the revisions together.)
Andrew Bennetts
andrew.bennetts at canonical.com
Thu Apr 2 13:02:22 BST 2009
Vincent Ladeuil wrote:
[...]
>
> Andrew> It also gets rid of the only use of get_revision_xml
> Andrew> in bzrlib outside of cmd_cat_revision and the test
> Andrew> suite. I also tidied the implementation of
> Andrew> get_revision_xml while I was glancing at it.
>
> Not good :-(
>
> LANG=C ./bzr selftest -s bzrlib.tests.blackbox.test_non_ascii.TestNonAscii.test_cat_revision
>
> succeeds (PQM setup striking again)
>
> LANG=en_US.UTF-8 ./bzr selftest -s bzrlib.tests.blackbox.test_non_ascii.TestNonAscii.test_cat_revision
>
> fails miserably ;-(
>
> The real fix being still a bit unclear (related to cStringIO
> encoding handling or its lack), the attached patch revert the
> get_revision_xml change.
Weird, but I see it too.
bb:approve
Thanks for the fix.
-Andrew.
More information about the bazaar
mailing list