comparison tests/run-tests.py @ 46067:cc0b332ab9fc

run-tests: stuff a `python3.exe` into the test bin directory on Windows Windows doesn't have `python3.exe` as part of the python.org distribution, and that broke every script with a shebang after c102b704edb5. Windows itself provides a `python3.exe` app execution alias[1], but it is some sort of reparse point that MSYS is incapable of handling[2]. When run by MSYS, it simply prints $ python3 -V - Cannot open That in turn caused every `hghave` check, and test that invokes shebang scripts directly, to fail. Rather than try to patch up every script call to be invoked with `$PYTHON` (and regress when non Windows developers forget), copying the executable into the test binary directory with the new name just works. Since this directory is prepended to the system PATH value, it also overrides the broken execution alias. (The `_tmpbindir` is used instead of `_bindir` because the latter causes python3.exe to be copied into the repo next to hg.exe when `test-run-tests.t` runs. Something runs with this version of the executable and subsequent runs of `run-tests.py` inside `test-run-tests.t` try to copy over it while it is in use, and fail. This avoids the failures and the clutter.) I didn't conditionalize this on py3 because `python3.exe` needs to be present (for the shebangs) even when running py2 tests. It shouldn't matter to these simple scripts, and I think the intention is to make the test runner use py3 always, even if testing a py2 build. For now, still supporting py2 is helping to clean up the mess that is py3 tests. [1] https://stackoverflow.com/a/57168165 [2] https://stackoverflow.com/questions/59148628/solved-unable-to-run-python-3-7-on-windows-10-permission-denied#comment104524397_59148666 Differential Revision: https://phab.mercurial-scm.org/D9543
author Matt Harbison <matt_harbison@yahoo.com>
date Mon, 07 Dec 2020 16:18:28 -0500
parents e4137c313e7f
children 9e785d940525
comparison
equal deleted inserted replaced
46066:e4137c313e7f 46067:cc0b332ab9fc
3464 except OSError as err: 3464 except OSError as err:
3465 # child processes may race, which is harmless 3465 # child processes may race, which is harmless
3466 if err.errno != errno.EEXIST: 3466 if err.errno != errno.EEXIST:
3467 raise 3467 raise
3468 else: 3468 else:
3469 # Windows doesn't have `python3.exe`, and MSYS cannot understand the
3470 # reparse point with that name provided by Microsoft. Copy the
3471 # current interpreter to PATH with that name so the shebang lines
3472 # work.
3473 if os.getenv('MSYSTEM'):
3474 shutil.copy(
3475 sys.executable,
3476 _bytes2sys(self._tmpbindir + b'/python3.exe'),
3477 )
3478
3469 exedir, exename = os.path.split(sysexecutable) 3479 exedir, exename = os.path.split(sysexecutable)
3470 vlog( 3480 vlog(
3471 "# Modifying search path to find %s as %s in '%s'" 3481 "# Modifying search path to find %s as %s in '%s'"
3472 % (exename, pyexename, exedir) 3482 % (exename, pyexename, exedir)
3473 ) 3483 )