================ @@ -0,0 +1,22 @@ +[ +{ + "directory": "$test_dir/build", ---------------- ilovepi wrote:
can't this just be "."? then everything is relative to that dir? if necessary you can `cd %t` to run commands in that dir. Subsequent test files run by `lit` will start from the build dir, so you don't have to cd back. It is generally nicer to have commands that can be run directly from the build dir, but its better to have easy to understand tests (and any tests at all) than to get bogged down with a minor detail like that. We can always file a bug to track that improvement as well. You can see something similar to what I've suggested in `clang-tools-extra/test/clang-tidy/infrastructure/run-clang-tidy.cpp`. Getting rid of `test_dir` should also mean you don't need that `sed` command. https://github.com/llvm/llvm-project/pull/93928 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits