fix: Another attempt to fix libcrypto.dylib loading issue (#112)

apache · Feb 26, 2024 · 7be5a18 · 7be5a18
1 parent 96dfccf
commit 7be5a18
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/.github/actions/setup-macos-builder/action.yaml b/.github/actions/setup-macos-builder/action.yaml
@@ -49,12 +49,13 @@ runs:
         unzip $PROTO_ZIP
         echo "$HOME/d/protoc/bin" >> $GITHUB_PATH
         export PATH=$PATH:$HOME/d/protoc/bin
-        # install openssl and setup DYLD_LIBRARY_PATH to work with libcrypto.dylib loading issues with x86_64 mac runners
-        # see PR https://github.com/apache/arrow-datafusion-comet/pull/55 for more details
+        # install openssl and setup DYLD_LIBRARY_PATH
         brew install openssl
-        OPENSSL_LIB_PATH=$(dirname `brew list openssl | grep 'lib/libcrypto.dylib'`)
+        OPENSSL_LIB_PATH=`brew --prefix openssl`/lib
         echo "openssl lib path is: ${OPENSSL_LIB_PATH}"
-        export DYLD_LIBRARY_PATH=$OPENSSL_LIB_PATH:$DYLD_LIBRARY_PATH
+        echo "DYLD_LIBRARY_PATH=$OPENSSL_LIB_PATH:$DYLD_LIBRARY_PATH" >> $GITHUB_ENV
+        # output the current status of SIP for later debugging
+        csrutil status || true
 
     - name: Install JDK ${{inputs.jdk-version}}
       uses: actions/setup-java@v4

diff --git a/.github/workflows/pr_build.yml b/.github/workflows/pr_build.yml
@@ -70,7 +70,7 @@ jobs:
   macos-test:
     strategy:
       matrix:
-        os: [macos-latest]
+        os: [macos-13]
         java_version: [8, 11, 17]
         test-target: [rust, java]
         is_push_event: