[Lldb-commits] [lldb] [lldb] Change directory creation logic in framework-header-fix (PR #158355)
    via lldb-commits 
    lldb-commits at lists.llvm.org
       
    Fri Sep 12 13:01:36 PDT 2025
    
    
  
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-lldb
Author: Alex Langford (bulbazord)
<details>
<summary>Changes</summary>
It's possible for this logic to fail if the build system runs this script in parallel. One instance could create the directory in between another instance's checking of its existence and attempt at creation.
Instead, always try to create it and ignore any FileExistsErrors.
rdar://160120161
---
Full diff: https://github.com/llvm/llvm-project/pull/158355.diff
1 Files Affected:
- (modified) lldb/scripts/framework-header-fix.py (+3-1) 
``````````diff
diff --git a/lldb/scripts/framework-header-fix.py b/lldb/scripts/framework-header-fix.py
index 36c5c67c59d36..3447dfc29a761 100755
--- a/lldb/scripts/framework-header-fix.py
+++ b/lldb/scripts/framework-header-fix.py
@@ -115,8 +115,10 @@ def main():
         unifdef_guards = ["-U" + guard for guard in args.unifdef_guards]
 
     # Create the framework's header dir if it doesn't already exist
-    if not os.path.exists(os.path.dirname(output_file_path)):
+    try:
         os.makedirs(os.path.dirname(output_file_path))
+    except FileExistsError:
+        pass
 
     if framework_version == "lldb_main":
         modify_main_includes(input_file_path, output_file_path)
``````````
</details>
https://github.com/llvm/llvm-project/pull/158355
    
    
More information about the lldb-commits
mailing list