From 82771daadd8d33eaf5087b43328458f970c6c378 Mon Sep 17 00:00:00 2001 From: Matteo Merli Date: Mon, 13 Aug 2018 16:56:23 -0700 Subject: [PATCH] Fixed linking of python wrapper to boost-python in MacOS build (#2366) --- pulsar-client-cpp/python/CMakeLists.txt | 4 +++- pulsar-client-cpp/python/pkg/osx/build.sh.template | 14 ++++++++++---- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/pulsar-client-cpp/python/CMakeLists.txt b/pulsar-client-cpp/python/CMakeLists.txt index 8e70f72ab4701..d506844482ea1 100644 --- a/pulsar-client-cpp/python/CMakeLists.txt +++ b/pulsar-client-cpp/python/CMakeLists.txt @@ -35,7 +35,9 @@ if("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang") set(CMAKE_SHARED_LIBRARY_CREATE_CXX_FLAGS "${CMAKE_SHARED_LIBRARY_CREATE_CXX_FLAGS} -undefined dynamic_lookup") endif() -set(PYTHON_WRAPPER_LIBS ${Boost_PYTHON_LIBRARY} ${Boost_PYTHON3_LIBRARY}) +# Try all possible boost-python variable namings +set(PYTHON_WRAPPER_LIBS ${Boost_PYTHON_LIBRARY} ${Boost_PYTHON3_LIBRARY} + ${Boost_PYTHON27-MT_LIBRARY} ${Boost_PYTHON37-MT_LIBRARY}) if (APPLE) target_link_libraries(_pulsar -Wl,-all_load pulsarStatic ${PYTHON_WRAPPER_LIBS}) diff --git a/pulsar-client-cpp/python/pkg/osx/build.sh.template b/pulsar-client-cpp/python/pkg/osx/build.sh.template index 71687ac4b4833..40b7420de8dcf 100755 --- a/pulsar-client-cpp/python/pkg/osx/build.sh.template +++ b/pulsar-client-cpp/python/pkg/osx/build.sh.template @@ -27,7 +27,7 @@ BOOST_VERSION=1.67 /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)" -brew install openssl git boost@$BOOST_VERSION pkg-config jsoncpp cmake protobuf260 log4cxx +brew install openssl git boost@$BOOST_VERSION pkg-config jsoncpp cmake protobuf260 brew link --force boost@$BOOST_VERSION @@ -40,8 +40,8 @@ fi if [ "$PYTHONVER" = "PYTHON3" ] then - brew install python boost-python3@BOOST_VERSION - brew link --force boost-python3@$BOOST_VERSION + brew install python boost-python3 + brew link --force boost-python3 fi brew link --force protobuf260 @@ -52,4 +52,10 @@ cd incubator-pulsar/pulsar-client-cpp cmake . -DBUILD_TESTS=OFF -DLINK_STATIC=ON make _pulsar -j8 cd python -python setup.py bdist_wheel + +if [ "$PYTHONVER" = "PYTHON2" ] +then + python2 setup.py bdist_wheel +else + python3 setup.py bdist_wheel +fi