We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
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
to use extended regex on OSX's sed requires the -E flag.
The text was updated successfully, but these errors were encountered:
the following patch fixes this and #21
diff --git a/CMakeLists.txt b/CMakeLists.txt index 4b6df3f..c964a58 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -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 --libs + 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}) @@ -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(
Sorry, something went wrong.
Fixes to build on MacOSX addresses Heeks#20 and Heeks#21
e7e0014
No branches or pull requests
to use extended regex on OSX's sed requires the -E flag.
The text was updated successfully, but these errors were encountered: