Patchwork D483: tests: fixed a bytes/unicode confusion in the test runner

login
register
mail settings
Submitter phabricator
Date Aug. 23, 2017, 1:58 p.m.
Message ID <c941d79acfc5dafc6b1f1f5aaf6bd208@localhost.localdomain>
Download mbox | patch
Permalink /patch/23229/
State Not Applicable
Headers show

Comments

phabricator - Aug. 23, 2017, 1:58 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG9e8b01d0654f: tests: fixed a bytes/unicode confusion in the test runner (authored by alex.gaynor).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D483?vs=1165&id=1198

REVISION DETAIL
  https://phab.mercurial-scm.org/D483

AFFECTED FILES
  tests/run-tests.py

CHANGE DETAILS




To: alex.gaynor, durin42, #hg-reviewers, indygreg
Cc: mercurial-devel

Patch

diff --git a/tests/run-tests.py b/tests/run-tests.py
--- a/tests/run-tests.py
+++ b/tests/run-tests.py
@@ -1459,7 +1459,7 @@ 
                         if not el.endswith(b" (?)\n"):
                             m = optline.match(el)
                             if m:
-                                conditions = [c for c in m.group(2).split(' ')]
+                                conditions = [c for c in m.group(2).split(b' ')]
 
                                 if self._hghave(conditions)[0]:
                                     # Don't append as optional line