[MERGE][1.14] Fix bug #355454 by casting to unicode.
John Arbash Meinel
john at arbash-meinel.com
Tue Apr 7 16:53:01 BST 2009
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Vincent Ladeuil wrote:
> This fixes the problem encountered by Bob while preparing the
> 1.14rc1 release.
>
> I'm a bit surprised that we need to use absolute paths here since
> the call sites should be mainly dirstate where relative paths
> should be cheaply available and may avoid that double handling.
>
> Anyway, this was blocking the release so we may work on a better
> fix later.
>
> Vincent
>
>
BB:comment
I think this is probably the wrong fix, but I'm not 100% so I'm
commenting rather than resubmitting.
If someone is passing in an abspath that isn't Unicode, then the caller
is not transmitting a proper path.
*paths* in memory should be unicode objects. The fix you supplied here
will break when whatever is calling this function supplies an encoded
string that has non-ascii characters.
I might be wrong, but I *think* this is exposing a bug in whoever is
calling these functions.
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iEYEARECAAYFAknbdtwACgkQJdeBCYSNAANOzQCcDCG3hXVeMRws07I0Sqnl4luV
NyQAn3SLoFNgCfsBSA3AbIuolbyvWnEr
=zd+j
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list