This is an automated email from the ASF dual-hosted git repository.

raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new e3150f2c68c GH-51487: [CI] Fix Emscripten wheel load failure by 
disabling llvm-strip on install (#51488)
e3150f2c68c is described below

commit e3150f2c68c17c3897b9d0c5aad26fd8e35883e6
Author: tadeja <[email protected]>
AuthorDate: Wed Sep 30 11:55:14 2026 +0200

    GH-51487: [CI] Fix Emscripten wheel load failure by disabling llvm-strip on 
install (#51488)
    
    ### Rationale for this change
    Fix #51487 - starting with LLVM 23 the required dylink.0 is not kept during 
CMake install strip. (see 
https://github.com/llvm/llvm-project/commit/52eb82e4f942db5e639c018c7757a1cca670333f)
    
    ### What changes are included in this PR?
    Disable wheel install stripping with `SKBUILD_INSTALL_STRIP=false`
    (applying alternative workaround llvm-strip --keep-section=dylink.0 would 
require more code changes)
    
    ### Are these changes tested?
    Verified by CI.
    
    ### Are there any user-facing changes?
    No.
    
    ### Was AI used for this PR?
    Just gpt-6-sol during initial analysis
    **PR code and description written by:**
    - [x] Human
    - [ ] AI
    
    **Reviewed before submission by:**
    - [x] Human
    - [ ] AI
    - [ ] Not reviewed
    * GitHub Issue: #51487
    
    Lead-authored-by: tadeja <[email protected]>
    Co-authored-by: Tadeja Kadunc <[email protected]>
    Co-authored-by: Raúl Cumplido <[email protected]>
    Signed-off-by: Raúl Cumplido <[email protected]>
---
 ci/scripts/python_build_emscripten.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/ci/scripts/python_build_emscripten.sh 
b/ci/scripts/python_build_emscripten.sh
index 061e7df7593..a0c97b8c2cd 100755
--- a/ci/scripts/python_build_emscripten.sh
+++ b/ci/scripts/python_build_emscripten.sh
@@ -44,5 +44,8 @@ if [ "${GITHUB_ACTIONS:-}" = "true" ]; then
 fi
 
 pushd "${python_build_dir}"
+# scikit-build-core strips unnecessary symbols by default.
+# LLVM 23 also removes the required dylink.0 section (llvm/llvm-project#180246)
+export SKBUILD_INSTALL_STRIP=false
 pyodide build
 popd

Reply via email to