intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-01-27 06:06:34 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert "[libc++] Use proper shell escaping in the executors"

Browse Source 
This reverts f8452ddfcc, which broke some bots. I'll figure out what's
wrong and commit it again.
This commit is contained in:
Louis Dionne
2020-04-17 17:06:17 -04:00
parent c858debebc
commit 7cb1aa9d93
2 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
libcxx/utils/run.py
View File

@@ -14,7 +14,6 @@ program's error code.
import argparse
import os
import pipes
import shutil
import subprocess
import sys
@@ -58,9 +57,8 @@ def main():
else:
shutil.copy2(dep, args.execdir)
# Run the command line with the given environment in the execution directory.
commandLine = (pipes.quote(x) for x in remaining)
return subprocess.call(' '.join(commandLine), cwd=args.execdir, env=env, shell=True)
# Run the executable with the given environment in the execution directory.
return subprocess.call(' '.join(remaining), cwd=args.execdir, env=env, shell=True)
finally:
shutil.rmtree(args.execdir)

4
libcxx/utils/ssh.py
View File

@@ -15,7 +15,6 @@ conformance test suite.
import argparse
import os
import pipes
import posixpath
import subprocess
import sys
@@ -98,11 +97,10 @@ def main():
# host by transforming the path of test-executables to their path in the
# temporary directory, where we know they have been copied when we handled
# test dependencies above.
commandLine = (pathOnRemote(x) if isTestExe(x) else x for x in commandLine)
remoteCommands += [
'cd {}'.format(tmp),
'export {}'.format(' '.join(args.env)),
' '.join(pipes.quote(x) for x in commandLine)
' '.join(pathOnRemote(x) if isTestExe(x) else x for x in commandLine)
]
# Finally, SSH to the remote host and execute all the commands.