D116: run-tests: check pygmentspresent in parseargs()

[martinvonz (Martin von Zweigbergk)]

martinvonz created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  Checking earlier if it's installed means we can remove the check
  later.

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  tests/run-tests.py

CHANGE DETAILS

diff --git a/tests/run-tests.py b/tests/run-tests.py
--- a/tests/run-tests.py
+++ b/tests/run-tests.py
@@ -420,7 +420,7 @@
     elif options.color == 'never':
         options.color = False
     else: # 'always', for testing purposes
-        options.color = True
+        options.color = pygmentspresent
 
     global useipv6
     if options.ipv6:
@@ -1650,7 +1650,7 @@
                 else:
                     self.stream.write('\n')
                     for line in lines:
-                        if self._options.color and pygmentspresent:
+                        if self._options.color:
                             line = pygments.highlight(
                                     line,
                                     lexers.DiffLexer(),



EMAIL PREFERENCES
  https://phab.mercurial-scm.org/settings/panel/emailpreferences/

To: martinvonz, #hg-reviewers
Cc: mercurial-devel