[all-commits] [llvm/llvm-project] f1b4c4: [lit] Fix `not` calling internal commands

[Joel E. Denny via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: f1b4c4bfd0b55c3c0abbc73e1159117f8533aca3
      https://github.com/llvm/llvm-project/commit/f1b4c4bfd0b55c3c0abbc73e1159117f8533aca3
  Author: Joel E. Denny <jdenny.ornl at gmail.com>
  Date:   2019-11-05 (Tue, 05 Nov 2019)

  Changed paths:
    M llvm/utils/lit/lit/TestRunner.py
    A llvm/utils/lit/tests/Inputs/shtest-env/env-calls-not-builtin.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/fail.py
    A llvm/utils/lit/tests/Inputs/shtest-not/lit.cfg
    A llvm/utils/lit/tests/Inputs/shtest-not/not-args-last-is-crash.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-args-nested-none.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-args-none.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-cd.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-colon.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-diff-with-crash.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-diff.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-echo.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-env-builtin.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-export.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-external.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-mkdir.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/not-calls-rm.txt
    A llvm/utils/lit/tests/Inputs/shtest-not/pass.py
    A llvm/utils/lit/tests/Inputs/shtest-not/print_environment.py
    M llvm/utils/lit/tests/shtest-env.py
    A llvm/utils/lit/tests/shtest-not.py

  Log Message:
  -----------
  [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