[libc++] NFC: Do not print the environment on remote hosts
authorLouis Dionne <ldionne@apple.com>
Wed, 6 May 2020 14:35:02 +0000 (10:35 -0400)
committerLouis Dionne <ldionne@apple.com>
Wed, 6 May 2020 15:33:13 +0000 (11:33 -0400)
Running `export` when there is no environment variable to export will
cause the environment on the remote host to be printed. We don't want
that, so don't run any `export` command on the host when there's no env.

libcxx/utils/ssh.py

index 6329cb4..50c6f45 100644 (file)
@@ -98,11 +98,10 @@ def main():
         # 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)),
-            subprocess.list2cmdline(commandLine)
-        ]
+        remoteCommands.append('cd {}'.format(tmp))
+        if 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.
         rc = subprocess.call(ssh(' && '.join(remoteCommands)))