diff options
author | Chris Bieneman <beanz@apple.com> | 2016-10-18 23:55:34 +0000 |
---|---|---|
committer | Chris Bieneman <beanz@apple.com> | 2016-10-18 23:55:34 +0000 |
commit | b47711ab274e2de40e014dc72d9d87f884a893d7 (patch) | |
tree | b8abad2d8fd6b796458db316b1f478e1b15bb603 /clang/test/SemaCUDA/function-overload.cu | |
parent | f50761f128950bfedbba7f9ab56d28e2b33f29dd (diff) | |
download | bcm5719-llvm-b47711ab274e2de40e014dc72d9d87f884a893d7.tar.gz bcm5719-llvm-b47711ab274e2de40e014dc72d9d87f884a893d7.zip |
When invoking Terminal, don't assume the default shell
Summary:
If a user has their shell set to a non-POSIX conferment shell the TestTerminal.py tests fail because the shell blurb constructed here may not work in their shell.
In my specific case fish-shell (The Friendly Interactive Shell - http://fishshell.com) does not support $?, it instead uses $status (because it is friendly).
This patch removes the assumption of your default shell by running the constructed bash command via "/bin/bash -c ...". This should be safer for users mutating their shell environment.
Reviewers: tfiala
Subscribers: joerg, lldb-commits
Differential Revision: https://reviews.llvm.org/D25750
llvm-svn: 284552
Diffstat (limited to 'clang/test/SemaCUDA/function-overload.cu')
0 files changed, 0 insertions, 0 deletions