[all-commits] [llvm/llvm-project] d98b9a: [libc++] NFC: Do not print the environment on remo...

Louis Dionne via All-commits all-commits at lists.llvm.org
Wed May 6 08:33:45 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: d98b9a41578f93685b1df16aa37ded893e74cf18
      https://github.com/llvm/llvm-project/commit/d98b9a41578f93685b1df16aa37ded893e74cf18
  Author: Louis Dionne <ldionne at apple.com>
  Date:   2020-05-06 (Wed, 06 May 2020)

  Changed paths:
    M libcxx/utils/ssh.py

  Log Message:
  -----------
  [libc++] NFC: Do not print the environment on remote hosts

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.


  Commit: 89bb9f8d78de4b85c2b889d75027783097a15b89
      https://github.com/llvm/llvm-project/commit/89bb9f8d78de4b85c2b889d75027783097a15b89
  Author: Louis Dionne <ldionne at apple.com>
  Date:   2020-05-06 (Wed, 06 May 2020)

  Changed paths:
    A libcxx/test/libcxx/selftest/newformat/shell-escape-pipes.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/narrow.stream.objects/cin.sh.cpp
    M libcxx/test/std/input.output/iostream.objects/wide.stream.objects/wcin.sh.cpp

  Log Message:
  -----------
  [libc++] Make sure the cin/wcin tests run on remote hosts

When running on remote hosts, we need the whole `echo 123 | %t.exe` command
to run on the remote host. Thus, we need to escape the pipe to make sure
the command is treated as `{ echo 123 | %t.exe } > %t.out` instead of
`{ echo 123 } | %t.exe > %t.out`m where only `echo 123` is run on the
remote host.


Compare: https://github.com/llvm/llvm-project/compare/6533c1da7fab...89bb9f8d78de


More information about the All-commits mailing list