[lit] Fix `not` calling internal commands
authorJoel E. Denny <jdenny.ornl@gmail.com>
Tue, 5 Nov 2019 15:05:10 +0000 (10:05 -0500)
committerJoel E. Denny <jdenny.ornl@gmail.com>
Tue, 5 Nov 2019 19:09:21 +0000 (14:09 -0500)
commitf1b4c4bfd0b55c3c0abbc73e1159117f8533aca3
tree48095a3da68bacea68e37e8e4331419296c669d0
parent4f12ba50bb28a1bf5c5131dcdf84121f04b0e584
[lit] Fix `not` calling internal commands

Without this patch, when using lit's internal shell, if `not` on a lit
RUN line calls `env`, `diff`, or any of the other in-process shell
builtins that lit implements, lit accidentally searches for the latter
as an external executable.  What's worse is that works fine when a
developer is testing on a platform where those executables are
available and behave as expected, but it then breaks on other
platforms.

`not` seems useful for some builtins, such as `diff`, so this patch
supports such uses.  `not --crash` does not seem useful for builtins,
so this patch diagnoses such uses.  In all cases, this patch ensures
shell builtins are found behind any sequence of `env` and `not`
commands.

`not` calling `env` calling an external command appears useful when
the `env` and external command are part of a lit substitution, as in
D65156.  This patch supports that by looking through any sequence of
`env` and `not` commands, building the environment from the `env`s,
and storing the `not`s.  The `not`s are then added back to the command
line without the `env`s to execute externally.  This avoids the need
to replicate the `not` implementation, in particular the `--crash`
option, in lit.

Reviewed By: rnk

Differential Revision: https://reviews.llvm.org/D66531
21 files changed:
llvm/utils/lit/lit/TestRunner.py
llvm/utils/lit/tests/Inputs/shtest-env/env-calls-not-builtin.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/fail.py [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/lit.cfg [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-args-last-is-crash.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-args-nested-none.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-args-none.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-cd.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-colon.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-diff-with-crash.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-diff.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-echo.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-env-builtin.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-export.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-external.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-mkdir.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/not-calls-rm.txt [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/pass.py [new file with mode: 0644]
llvm/utils/lit/tests/Inputs/shtest-not/print_environment.py [new file with mode: 0644]
llvm/utils/lit/tests/shtest-env.py
llvm/utils/lit/tests/shtest-not.py [new file with mode: 0644]