durin42 added a comment.
I also popped https://phab.mercurial-scm.org/D964 out of the stack until I
can address review feedback. :)
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D965
To: durin42, #hg-reviewers, pulkit, indygreg, yuja
Cc: yuja, indygreg, pulkit,
indygreg added a comment.
Bleh. I missed yuya's comment before doing a delayed push. Feel free to drop
this from hg-committed or address as a follow-up.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D965
To: durin42, #hg-reviewers, pulkit, indygreg, yuja
Cc: yu
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG4296bbb19410: templatefilters: be sure we always feed
cgi.escape a str (authored by durin42, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D965?vs
yuja requested changes to this revision.
yuja added a comment.
This revision now requires changes to proceed.
sysstr -> sysbytes can't be round-trip if non-ascii characters are involved.
Perhaps we can just vendor cgi.escape() and add b'' to every string.
REPOSITORY
rHG Mercurial
REVIS
indygreg accepted this revision.
indygreg added a comment.
This revision is now accepted and ready to land.
Oof. It is really tempting to roll our own function that doesn't require this
ugliness. But that can be a follow-up.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial
pulkit accepted this revision.
pulkit added a comment.
Checked docs and made sure `cgi.escape` on Python 3 accepts str only.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D965
To: durin42, #hg-reviewers, pulkit
Cc: pulkit, mercurial-devel
___
durin42 created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D965
AFFECTED FILES
mercurial/templatefilters.py
CHANGE DETAILS
diff --git a/mercurial/templatefilters