comparison tests/run-tests.py @ 41198:4f0ae5c64c1b

tests: make HGCATAPULTSERVERPIPE imply HGTESTCATAPULTSERVERPIPE I had attempted to do this before, but missed this case. This makes it so that one can do the following to get catapult traces that include both the .t test name (and non-hg commands run by that .t test) *and* the hg-internal tracing, in one trace: HGCATAPULTSERVERPIPE=/tmp/catapult.pipe run-tests.py <args> Without this change, we need to specify both `HG{,TEST}CATAPULTSERVERPIPE`; if we specify just the TEST one, we only get the .t tests (no hg-internals), which is working as intended. If we specify the non-TEST one, we only get the hg-internals (not the rest of the .t test), which was not intended. If you want to restore the previous behavior (just hg internals, not the stuff from the .t tests), run like: HGTESTCATAPULTSERVERPIPE=/dev/null \ HGCATAPULTSERVERPIPE=/tmp/catapult.pipe \ run-tests.py <args> Differential Revision: https://phab.mercurial-scm.org/D5569
author Kyle Lippincott <spectral@google.com>
date Thu, 10 Jan 2019 19:25:07 -0800
parents 53327bfbf35d
children 876494fd967d
comparison
equal deleted inserted replaced
41197:172296c6db91 41198:4f0ae5c64c1b
1407 script.append(b'echo %s %d $?\n' % (salt, line)) 1407 script.append(b'echo %s %d $?\n' % (salt, line))
1408 activetrace = [] 1408 activetrace = []
1409 session = str(uuid.uuid4()) 1409 session = str(uuid.uuid4())
1410 if PYTHON3: 1410 if PYTHON3:
1411 session = session.encode('ascii') 1411 session = session.encode('ascii')
1412 hgcatapult = os.getenv('HGTESTCATAPULTSERVERPIPE') 1412 hgcatapult = os.getenv('HGTESTCATAPULTSERVERPIPE') or \
1413 os.getenv('HGCATAPULTSERVERPIPE')
1413 def toggletrace(cmd=None): 1414 def toggletrace(cmd=None):
1414 if not hgcatapult or hgcatapult == os.devnull: 1415 if not hgcatapult or hgcatapult == os.devnull:
1415 return 1416 return
1416 1417
1417 if activetrace: 1418 if activetrace: