[libcxx] [test] Remove --env PATH from the static clangcl config
authorMartin Storsjö <martin@martin.st>
Tue, 9 Aug 2022 08:36:51 +0000 (11:36 +0300)
committerMartin Storsjö <martin@martin.st>
Fri, 19 Aug 2022 08:40:58 +0000 (11:40 +0300)
commite78223e79efc886ef6f0ea5413deab3737d6d63b
tree32163d83576d1df36201548aac1bdb6cff47eeff
parent3fd4213059a4ea9453809aeccd1bfc7d115d24b2
[libcxx] [test] Remove --env PATH from the static clangcl config

The PATH is set in order to be able to find the tested DLL at runtime.
When linking statically, it's not necessary to set the PATH.

Setting PATH in the executor has the downside that it clears the
existing path (it's not prepended/appended to it), which means
that the executed tools can't find other tools - which sets the
executor-has-no-bash flag.

By removing the unnecessary setting of PATH, we have a properly
working bash even when wrapped by the executor, which gets rid
of the executor-has-no-bash flag in this test configuration, which
makes 9 more testcases be executed.

Differential Revision: https://reviews.llvm.org/D131481
libcxx/test/configs/llvm-libc++-static-clangcl.cfg.in