Skip to content

Commit

Permalink
bootstrap.py: Always use .exe for Windows
Browse files Browse the repository at this point in the history
This ensure that it will run the Windows executable if other files in the directory (such as Linux executables) have the same file name minus the extension.
  • Loading branch information
ChrisDenton committed Jul 4, 2022
1 parent a3beeaa commit b1541ff
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/bootstrap/bootstrap.py
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,10 @@ def run(args, verbose=False, exception=False, is_bootstrap=False, **kwargs):
if verbose:
print("running: " + ' '.join(args))
sys.stdout.flush()
# Ensure that the .exe is used on Windoows just in case a Linux ELF has been
# compiled in the same directory.
if os.name == 'nt' and not args[0].endswith('.exe'):
args[0] += '.exe'
# Use Popen here instead of call() as it apparently allows powershell on
# Windows to not lock up waiting for input presumably.
ret = subprocess.Popen(args, **kwargs)
Expand Down

0 comments on commit b1541ff

Please sign in to comment.