[PATCH] D65697: [lit] Fix internal env calling env

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

In D65697#1640136 <https://reviews.llvm.org/D65697#1640136>, @jdenny wrote:

> In D65697#1640109 <https://reviews.llvm.org/D65697#1640109>, @rnk wrote:
>
> > Most LLVM test suites are configured to use the internal shell on Windows. I think what's happening is that users happen to have an external copy of `diff` that gets called when it appears in a pipeline, or they are prefixed by `not`. The external `diff` is called, and things "work out" and we never noticed.
>
>
> Are these patches going to break those tests on Windows?  We now see `diff` anywhere in a pipeline, and we reject its appearance in a pipeline.


`grep -r -l 'RUN:.* diff ' llvm/test clang/test` gets 240 hits.  Searching on `| diff` gets 41, mostly in clang/test/Analysis.  So not a super common pattern, but if it's straightforward to build a Python 'diff' that handles pipes, that would be less invasive, and piping does have a minor speed advantage.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65697/new/

https://reviews.llvm.org/D65697