Skip to content

Conversation

@wesm
Copy link
Member

@wesm wesm commented Jun 27, 2019

Using the Homebrew provided c-ares results in a shared library dependency. I think it's easier to statically link this than to bundle it

$ otool -L /Users/wesm/miniconda/envs/wheel-test-3.7/lib/python3.7/site-packages/pyarrow/libarrow_flight.14.dylib 
/Users/wesm/miniconda/envs/wheel-test-3.7/lib/python3.7/site-packages/pyarrow/libarrow_flight.14.dylib:
	@rpath/libarrow_flight.14.dylib (compatibility version 14.0.0, current version 14.0.0)
	@rpath/libarrow.14.dylib (compatibility version 14.0.0, current version 14.0.0)
	/usr/local/opt/c-ares/lib/libcares.2.dylib (compatibility version 6.0.0, current version 6.0.0)
	/usr/local/opt/openssl/lib/libssl.1.0.0.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/local/opt/openssl/lib/libcrypto.1.0.0.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 307.5.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1238.50.2)

@wesm
Copy link
Member Author

wesm commented Jun 27, 2019

cc @pcmoritz. Is it reasonable to assume that macOS users will have OpenSSL on their systems?

@wesm
Copy link
Member Author

wesm commented Jun 27, 2019

Here's a Crossbow build:

https://travis-ci.org/wesm/crossbow/builds/551375423

I'll have a look later to see if it passed

@wesm
Copy link
Member Author

wesm commented Jun 27, 2019

OK, the patch was successful. Merging

$ otool -L /Users/wesm/miniconda/envs/wheel-test-3.7/lib/python3.7/site-packages/pyarrow/libarrow_flight.14.dylib
/Users/wesm/miniconda/envs/wheel-test-3.7/lib/python3.7/site-packages/pyarrow/libarrow_flight.14.dylib:
	@rpath/libarrow_flight.14.dylib (compatibility version 14.0.0, current version 14.0.0)
	@rpath/libarrow.14.dylib (compatibility version 14.0.0, current version 14.0.0)
	/usr/local/opt/openssl/lib/libssl.1.0.0.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/local/opt/openssl/lib/libcrypto.1.0.0.dylib (compatibility version 1.0.0, current version 1.0.0)
	/usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 307.5.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1238.50.2)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant