On Tue, 21 Mar 2017 15:13:01 -0400, Augie Fackler wrote: > # HG changeset patch > # User Augie Fackler <au...@google.com> > # Date 1489900459 14400 > # Sun Mar 19 01:14:19 2017 -0400 > # Node ID 63466a54ec9e266031cce12fbd699ccfccd40738 > # Parent a4745fd9219ed5b408bfc0403a4a8e6acd41df6c > revsetlang: work around repr() returning unicode on py3 > > I'm not confident in my choice of "encode to utf8" here: it seems > reasonable that we could expect the revset engine to do the right > thing there. On the other hand these strings might have encoding > weirdness to begin with if there's `branch(some-weird-byte-sequence)`.
> --- a/mercurial/revsetlang.py > +++ b/mercurial/revsetlang.py > @@ -607,7 +607,11 @@ def formatspec(expr, *args): > ''' > > def quote(s): > - return repr(str(s)) > + r = repr(bytes(s)) > + if pycompat.ispy3: > + r = r[1:] # strip off b prefix > + return r.encode('utf-8') utf-8 is wrong, but it would be okay as repr(bytes) wouldn't contain non-ascii characters. That said, it'll be better to use util.escapestr(). "'" + util.escapestr(s) + "'" _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel