[libc++] Use proper shell escaping in the executors
This was originally committed as f8452ddfcc33 and reverted in 7cb1aa9d9368.
The issue was that shell builtins were being escaped too, and apparently
Bash won't execute a builtin when it is quoted e.g. '!'. Instead, it
thinks it's a command and it can't find it.
Re-committing the change with that issue fixed.
Cr-Mirrored-From: https://chromium.googlesource.com/external/github.com/llvm/llvm-project
Cr-Mirrored-Commit: 5eb8d45ab5b85cb3a2edfe995cbf1d4b1beae462
diff --git a/utils/ssh.py b/utils/ssh.py
index c7d8c97..4bb983e 100644
--- a/utils/ssh.py
+++ b/utils/ssh.py
@@ -97,10 +97,11 @@
# 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(pathOnRemote(x) if isTestExe(x) else x for x in commandLine)
+ subprocess.list2cmdline(commandLine)
]
# Finally, SSH to the remote host and execute all the commands.