Mercurial > hg
diff tests/run-tests.py @ 34040:85bfd6a0bdbf
run-tests: extract prefix of bisect commands to a variable
This does not change any logic.
Differential Revision: https://phab.mercurial-scm.org/D577
author | Jun Wu <quark@fb.com> |
---|---|
date | Wed, 02 Aug 2017 21:01:38 -0700 |
parents | 72b23c9452d6 |
children | 40313c63da87 |
line wrap: on
line diff
--- a/tests/run-tests.py Wed Aug 02 21:01:38 2017 -0700 +++ b/tests/run-tests.py Wed Aug 02 21:01:38 2017 -0700 @@ -2070,15 +2070,16 @@ savetimes(self._runner._outputdir, result) if failed and self._runner.options.known_good_rev: + bisectcmd = ['hg', 'bisect'] def nooutput(args): p = subprocess.Popen(args, stderr=subprocess.STDOUT, stdout=subprocess.PIPE) p.stdout.read() p.wait() for test, msg in result.failures: - nooutput(['hg', 'bisect', '--reset']), - nooutput(['hg', 'bisect', '--bad', '.']) - nooutput(['hg', 'bisect', '--good', + nooutput(bisectcmd + ['--reset']), + nooutput(bisectcmd + ['--bad', '.']) + nooutput(bisectcmd + ['--good', self._runner.options.known_good_rev]) # TODO: we probably need to forward more options # that alter hg's behavior inside the tests. @@ -2088,7 +2089,7 @@ opts += ' --with-hg=%s ' % shellquote(withhg) rtc = '%s %s %s %s' % (sys.executable, sys.argv[0], opts, test) - sub = subprocess.Popen(['hg', 'bisect', '--command', rtc], + sub = subprocess.Popen(bisectcmd + ['--command', rtc], stderr=subprocess.STDOUT, stdout=subprocess.PIPE) data = sub.stdout.read()