D3633: py3: use bytes in tests/printenv.py
indygreg (Gregory Szorc)
phabricator at mercurial-scm.org
Mon May 21 14:29:10 EDT 2018
indygreg accepted this revision.
indygreg added a comment.
This revision is now accepted and ready to land.
This one worries me a bit because environment variable values can contain non-ASCII. However, the changed code only operates on `HG_` variables and I //think// those variables are well-sanitized and should be ASCII safe.
Also, `os.environ` is `str`/`bytes` on Python 2 and the `.encode()` is nonsensical. But Python 2 doesn't care, so meh.
Since this is testing code, the barrier for change is low. And the failure mode is pretty obvious. So I'm OK landing this now and fixing later if it causes problems.
To: pulkit, #hg-reviewers, indygreg
Cc: indygreg, mercurial-devel
More information about the Mercurial-devel