[libcxx-commits] [PATCH] D97452: [libcxx] [test] Pass some windows environment variables through to test processes

[Martin Storsjö via Phabricator via libcxx-commits]

mstorsjo added a comment.

In D97452#2587321 <https://reviews.llvm.org/D97452#2587321>, @curdeius wrote:

> LGTM. But I believe that the long term solution is to run all the tests in the internal lit shell.

Yes, but I believe that aspect is orthogonal to this script. Currently the lit test driver runs the test script (which consists of two lines, one "clang++" to compile the test and one which invokes the run.py script with python to execute the built test executable) with bash, which maybe could/should be changed to use the lit internal shell. That still keeps the run.py wrapper script for all the purposes it serves right now (codesign, setting up other test specific env variables etc).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97452