[clang] [clang][lit] mkdir before mkstemp in is_filesystem_case_insensitive() (PR #131036)

via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 12 14:45:36 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Arthur Eubanks (aeubanks)

<details>
<summary>Changes</summary>

In the CMake build test_exec_root already exists here, but not in the gn build, which causes this to fail.

---
Full diff: https://github.com/llvm/llvm-project/pull/131036.diff


1 Files Affected:

- (modified) clang/test/lit.cfg.py (+1) 


``````````diff
diff --git a/clang/test/lit.cfg.py b/clang/test/lit.cfg.py
index 9820ddd1f14af..074167fa6cad3 100644
--- a/clang/test/lit.cfg.py
+++ b/clang/test/lit.cfg.py
@@ -253,6 +253,7 @@ def have_host_clang_repl_cuda():
 
 
 def is_filesystem_case_insensitive():
+    os.mkdir(config.test_exec_root)
     handle, path = tempfile.mkstemp(prefix="case-test", dir=config.test_exec_root)
     isInsensitive = os.path.exists(
         os.path.join(os.path.dirname(path), os.path.basename(path).upper())

``````````

</details>


https://github.com/llvm/llvm-project/pull/131036


More information about the cfe-commits mailing list