This is an archive of the discontinued LLVM Phabricator instance.

Fix environment variable passing in libc++'s `SSHExecutor` `lit` utility

Authored by wash on Aug 8 2019, 10:21 AM.



Quote the value of environment variables when passing them to the SSH client in SSHExecutor in libc++'s lit utilities. Without the quotes, an environment variable like FOO="buzz bar" gets passed incorrectly like this, ssh env FOO=buzz bar, which causes bar to be treated as a command to run, not part of the environment variable value.

We ran into this when using SSHExecutor to do bringup of our CUDA libcu++ port on an embedded aarch64 system.

Diff Detail


Event Timeline

wash created this revision.Aug 8 2019, 10:21 AM
ldionne accepted this revision.Aug 8 2019, 10:32 AM
This revision is now accepted and ready to land.Aug 8 2019, 10:32 AM
This revision was automatically updated to reflect the committed changes.