diff options
| author | Martin Storsjö <martin@martin.st> | 2023-04-27 10:11:05 +0300 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2023-04-27 19:25:59 +0300 |
| commit | eb5f9a5a521ba1786a2648b2d9b36b506ebd26e4 (patch) | |
| tree | d07dae05efd5330bc3dbb9011482870a4528d492 /libcxx/utils | |
| parent | 1ec4e139efb474d98b38c4f6c5f0ee3750b21c69 (diff) | |
| download | llvm-eb5f9a5a521ba1786a2648b2d9b36b506ebd26e4.tar.gz | |
[libcxx] [test] Unbreak passing multiple env variables in ssh.py
No test actually does this, but this makes the option behave like
the corresponding one in run.py.
This was broken by commit b8b23aa80eefe84187d6ba364d06496c90c53bdb
(https://reviews.llvm.org/D99242) which introduced quoting; instead
of quoting the whole space separated list, quote each individual
argument.
Differential Revision: https://reviews.llvm.org/D149319
Diffstat (limited to 'libcxx/utils')
| -rwxr-xr-x | libcxx/utils/ssh.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libcxx/utils/ssh.py b/libcxx/utils/ssh.py index 6c1d706984dd..281ff10b52bc 100755 --- a/libcxx/utils/ssh.py +++ b/libcxx/utils/ssh.py @@ -109,7 +109,8 @@ def main(): commandLine = (pathOnRemote(x) if isTestExe(x) else x for x in commandLine) remoteCommands.append('cd {}'.format(tmp)) if args.env: - remoteCommands.append('export {}'.format(cmd_quote(' '.join(args.env)))) + env = list(map(cmd_quote, args.env)) + remoteCommands.append('export {}'.format(' '.join(args.env))) remoteCommands.append(subprocess.list2cmdline(commandLine)) # Finally, SSH to the remote host and execute all the commands. |