Skip to content

Commit

Permalink
Setting the correct PYTHON_SITE_PACKAGES_PREFIX for apple
Browse files Browse the repository at this point in the history
  • Loading branch information
@jbeezley
jbeezley committed Jan 28, 2015
1 parent 804abe8 commit 03e3a6b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion CMake/cdat_modules/python_pkg.cmake
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@ set(CDAT_OS_XTRA_PATH "")
set(PYTHON_SITE_PACKAGES_PREFIX ${CMAKE_INSTALL_PREFIX})
set(PYTHON_SITE_PACKAGES ${PYTHON_SITE_PACKAGES_PREFIX}/lib/python${PYVER}/site-packages)
if (APPLE)
set(PYTHON_SITE_PACKAGES ${PYTHON_SITE_PACKAGES_PREFIX}/Library/Frameworks/Python.framework/Versions/${PYVER}/lib/python${PYVER}/site-packages)
set(PYTHON_SITE_PACKAGES_PREFIX ${CMAKE_INSTALL_PREFIX}/Library/Frameworks/Python.framework/Versions/${PYVER})
set(PYTHON_SITE_PACKAGES ${PYTHON_SITE_PACKAGES_PREFIX}/lib/python${PYVER}/site-packages)
endif()
set(PYTHONPATH ${PYTHON_SITE_PACKAGES})

Expand Down

2 comments on commit 03e3a6b

@remram44
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couldn't the if be one line earlier so that you don't repeat line 17=20?

@jbeezley
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, revised in latest commit.

Please sign in to comment.