Skip to content

Fixes to build on MacOSX addresses #20 and #21 #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
silicontrip wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixes to build on MacOSX addresses #20 and #21 #22

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e7e0014
Fixes to build on MacOSX addresses #20 and #21
Jul 8, 2017
edeec3b
Changes for python-config flags
silicontrip Jul 14, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 16 additions & 1 deletion CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,16 +27,30 @@ if (NOT BUILD_TYPE)
set(CMAKE_BUILD_TYPE Debug)
endif(NOT BUILD_TYPE)

if (APPLE)
set(CMAKE_MACOSX_RPATH 1)
set(SED_COMMAND sed -E)
else(APPLE)
set(SED_COMMAND sed -r)
endif()

# this figures out the Python include directories and adds them to the
# header file search path
execute_process(
COMMAND python-config --includes
COMMAND sed -r "s/-I//g; s/ +/;/g"
COMMAND ${SED_COMMAND} "s/-I//g; s/ +/;/g"
COMMAND tr -d '\n'
OUTPUT_VARIABLE Python_Includes
)
message(STATUS "Python include dir:" ${Python_Includes})

execute_process(
COMMAND python-config --ldflags
COMMAND tr -d '\n'
OUTPUT_VARIABLE Python_Link_Flags
)
message(STATUS "Python link flags:" ${Python_Link_Flags})

include_directories(${Python_Includes})
include_directories(${CMAKE_CURRENT_SOURCE_DIR})

Expand Down Expand Up @@ -166,6 +180,7 @@ else (USE_CLIPPER_FOR_PYTHON)
endif (USE_CLIPPER_FOR_PYTHON)
target_link_libraries(area ${Boost_LIBRARIES})
set_target_properties(area PROPERTIES PREFIX "")
set_target_properties(area PROPERTIES LINK_FLAGS "${Python_Link_Flags}")

# this figures out where to install the Python modules
execute_process(
Expand Down