[libcxx-commits] [PATCH] D99241: [libcxx] [test] Don't add dirs from the LIB env var to PATH

[Martin Storsjö via Phabricator via libcxx-commits]

mstorsjo added a comment.

In D99241#2656852 <https://reviews.llvm.org/D99241#2656852>, @amccarth wrote:

> Adding everything in LIB to PATH is not a normal thing to do in Windows, so I would support this patch to remove that hack.  If we knew what the original problem was, we could probably find a better solution.
>
> When you run the tests, does anything differ with and without this patch?

Nothing differs when it comes to actually executing/passing tests. The only visible difference is that the executor scripts are passed fewer paths via the `--env` option.

So my guess is that it was an attempt at fixing something, done at the same time as a number of other changes, without verifying that this particular change actually had any effect.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99241