https://github.com/boomanaiden154 updated https://github.com/llvm/llvm-project/pull/170676
>From ca508323f04448214e6a3dd02fb750f332683e3f Mon Sep 17 00:00:00 2001 From: Aiden Grossman <[email protected]> Date: Thu, 4 Dec 2025 15:01:44 +0000 Subject: [PATCH] Reapply "[clangd] Enable lit internal shell by default" This reverts commit 4cfbc44ebe26692c209655c37aeb0b6cbf1d479b. There was an issue setting up the PATH variable in system-include-extractor.test which prevented chmod from being resolved. --- clang-tools-extra/clangd/test/lit.cfg.py | 15 ++++++++++++++- .../clangd/test/system-include-extractor.test | 2 +- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/clang-tools-extra/clangd/test/lit.cfg.py b/clang-tools-extra/clangd/test/lit.cfg.py index 05a0f5e7383e9..bd721c41861b2 100644 --- a/clang-tools-extra/clangd/test/lit.cfg.py +++ b/clang-tools-extra/clangd/test/lit.cfg.py @@ -1,3 +1,5 @@ +import os + import lit.llvm import lit.util @@ -5,10 +7,21 @@ lit.llvm.llvm_config.clang_setup() lit.llvm.llvm_config.use_default_substitutions() +# TODO: Consolidate the logic for turning on the internal shell by default for all LLVM test suites. +# See https://github.com/llvm/llvm-project/issues/106636 for more details. +# +# We prefer the lit internal shell which provides a better user experience on failures +# and is faster unless the user explicitly disables it with LIT_USE_INTERNAL_SHELL=0 +# env var. +use_lit_shell = True +lit_shell_env = os.environ.get("LIT_USE_INTERNAL_SHELL") +if lit_shell_env: + use_lit_shell = lit.util.pythonize_bool(lit_shell_env) + config.name = "Clangd" config.suffixes = [".test"] config.excludes = ["Inputs"] -config.test_format = lit.formats.ShTest(not lit.llvm.llvm_config.use_lit_shell) +config.test_format = lit.formats.ShTest(not use_lit_shell) config.test_source_root = config.clangd_source_dir + "/test" config.test_exec_root = config.clangd_binary_dir + "/test" diff --git a/clang-tools-extra/clangd/test/system-include-extractor.test b/clang-tools-extra/clangd/test/system-include-extractor.test index 3314be806a801..861d2e188a1f8 100644 --- a/clang-tools-extra/clangd/test/system-include-extractor.test +++ b/clang-tools-extra/clangd/test/system-include-extractor.test @@ -5,7 +5,7 @@ # Create a bin directory to store the mock-driver and add it to the path # RUN: mkdir -p %t.dir/bin -# RUN: %python -c "print(__import__('os').environ['PATH'])" > %t.path +# RUN: %python -c "print(__import__('os').environ['PATH'])" | tr -d '\n' > %t.path # RUN: export PATH=%t.dir/bin:%{readfile:%t.path} # Generate a mock-driver that will print %temp_dir%/my/dir and # %temp_dir%/my/dir2 as include search paths. _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
