Skip to content

Commit

Permalink
Check if likely-to-be-used Numpy include directory is from virtual en…
Browse files Browse the repository at this point in the history
…vironment

Addresses python-controlgh-193, in that the user gets an error.
  • Loading branch information
roryyorke authored and KybernetikJo committed Aug 25, 2023
1 parent e65ec6d commit a29514b
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 a29514b

Please sign in to comment.