Skip to content

Commit

Permalink
Merge pull request #197 from roryyorke/gh193
Browse files Browse the repository at this point in the history
Check if likely-to-be-used Numpy include directory is from virtual env
  • Loading branch information
bnavigator authored Aug 24, 2023
2 parents 8ae9c25 + d3ef095 commit 0f35d08
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,18 @@ message(STATUS "F2PY headers included from: ${F2PY_INCLUDE_DIRS}")
message(STATUS "LAPACK: ${LAPACK_LIBRARIES}")
message(STATUS "BLAS: ${BLAS_LIBRARIES}")

# https://github.com/python-control/Slycot/issues/193
if((EXISTS "${Python_INCLUDE_DIRS}/numpy")
AND (NOT ("${Python_INCLUDE_DIRS}/numpy" EQUAL "${Python_NumPy_INCLUDE_DIRS}")))

message(FATAL_ERROR
"Python include directory has a numpy sub-directory,
${Python_INCLUDE_DIRS}/numpy,
which is different from Numpy include directory
${Python_NumPy_INCLUDE_DIRS}.
You're probably building in a virtual environment, in which case
uninstall numpy from the base environment and try again.")

endif()

add_subdirectory(slycot)

0 comments on commit 0f35d08

Please sign in to comment.