llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: nerix (Nerixyz)

<details>
<summary>Changes</summary>

On Windows, the post build command would open the script in the default editor, 
since it doesn't know about shebangs. This effectively adds `python3` in front 
of the command.

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


1 Files Affected:

- (modified) lldb/source/API/CMakeLists.txt (+1-1) 


``````````diff
diff --git a/lldb/source/API/CMakeLists.txt b/lldb/source/API/CMakeLists.txt
index 2772aa920411f..f4a323aaf267c 100644
--- a/lldb/source/API/CMakeLists.txt
+++ b/lldb/source/API/CMakeLists.txt
@@ -326,7 +326,7 @@ foreach(header
 endforeach()
 
 add_custom_command(TARGET liblldb POST_BUILD
-  COMMAND ${LLDB_SOURCE_DIR}/scripts/version-header-fix.py 
${LLDB_SOURCE_DIR}/include/lldb/lldb-defines.h 
${lldb_header_staging_dir}/lldb-defines.h ${LLDB_VERSION_MAJOR} 
${LLDB_VERSION_MINOR} ${LLDB_VERSION_PATCH}
+  COMMAND "${Python3_EXECUTABLE}" 
${LLDB_SOURCE_DIR}/scripts/version-header-fix.py 
${LLDB_SOURCE_DIR}/include/lldb/lldb-defines.h 
${lldb_header_staging_dir}/lldb-defines.h ${LLDB_VERSION_MAJOR} 
${LLDB_VERSION_MINOR} ${LLDB_VERSION_PATCH}
 )
 add_custom_target(liblldb-header-staging DEPENDS ${lldb_staged_headers})
 add_dependencies(liblldb liblldb-header-staging)

``````````

</details>


https://github.com/llvm/llvm-project/pull/144217
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to