You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2019/07/13 17:02:19 UTC

[arrow] 24/43: ARROW-5874: [Python] Fix macOS wheels to depend on system or Homebrew OpenSSL

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

wesm pushed a commit to branch maint-0.14.x
in repository https://gitbox.apache.org/repos/asf/arrow.git

commit 0f4e0d603a14279763ef06467b3cd208fbcefba1
Author: Krisztián Szűcs <sz...@gmail.com>
AuthorDate: Mon Jul 8 19:51:01 2019 +0200

    ARROW-5874: [Python] Fix macOS wheels to depend on system or Homebrew OpenSSL
    
    The wheel build and test build succeeds, just the deployments fails because of removed openssl for testing: https://travis-ci.org/ursa-labs/crossbow/builds/555822505
    
    The currently running build should pass: https://travis-ci.org/ursa-labs/crossbow/builds/555849713
    although we have a bunch of warnings `<lib> was built for newer OSX version (10.12) than being linked (10.9)` for the dependencies coming from brew.
    
    cc @xhochy
    
    Author: Krisztián Szűcs <sz...@gmail.com>
    
    Closes #4823 from kszucs/osx-wheel-openssl and squashes the following commits:
    
    eb45ae47d <Krisztián Szűcs> reinstall openssl
    ca8d177a6 <Krisztián Szűcs> ignore deps
    be4ab0f77 <Krisztián Szűcs> openssl
---
 dev/tasks/python-wheels/osx-build.sh   | 1 +
 dev/tasks/python-wheels/travis.osx.yml | 4 +++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/dev/tasks/python-wheels/osx-build.sh b/dev/tasks/python-wheels/osx-build.sh
index b299646..517a63a 100755
--- a/dev/tasks/python-wheels/osx-build.sh
+++ b/dev/tasks/python-wheels/osx-build.sh
@@ -138,6 +138,7 @@ function build_wheel {
           -DgRPC_SOURCE=SYSTEM \
           -Dc-ares_SOURCE=BUNDLED \
           -DARROW_PROTOBUF_USE_SHARED=OFF \
+          -DOPENSSL_USE_STATIC_LIBS=ON  \
           -DMAKE=make \
           ..
     make -j5
diff --git a/dev/tasks/python-wheels/travis.osx.yml b/dev/tasks/python-wheels/travis.osx.yml
index 980d122..8bea257 100644
--- a/dev/tasks/python-wheels/travis.osx.yml
+++ b/dev/tasks/python-wheels/travis.osx.yml
@@ -68,11 +68,13 @@ install:
 
   # test the built wheels, remove llvm and grpc dependencies to ensure
   # things are properly statically-linked
-  - brew uninstall llvm@7 grpc c-ares
+  - brew uninstall --ignore-dependencies llvm@7 grpc c-ares openssl
   - install_run arrow
 
   # move built wheels to a top level directory
   - mv -v arrow/python/dist/* dist/
+  # reinstall openssl because travis' deployment script depends on it
+  - brew install openssl
 
 deploy:
   provider: releases