[libcxx-commits] [PATCH] D97452: [libcxx] [test] Pass some windows environment variables through to test processes
Martin Storsjö via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Feb 25 04:31:03 PST 2021
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).
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libcxx-commits