Fixed issue that was causing the wrong cython executable from being c…

…alled when building the editable wheel
MintonGroup · Sep 18, 2023 · ba23e4f · ba23e4f
1 parent f341a78
commit ba23e4f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/buildscripts/install_editable_debug.sh b/buildscripts/install_editable_debug.sh
@@ -8,7 +8,7 @@ cd ${ROOT_DIR}
 python3 -m venv ${VENV_DIR}
 . ${VENV_DIR}/bin/activate
 python3 -m pip install --upgrade pip 
-pip install scikit-build-core pyproject-metadata pathspec ninja 
+pip install scikit-build-core pyproject-metadata pathspec ninja cython
 pip install --config-settings=editable.rebuild=true \
             --config-settings=build-dir="build/{wheel_tag}" \
             --config-settings=cmake.build-type="Debug" \

diff --git a/swiftest/CMakeLists.txt b/swiftest/CMakeLists.txt
@@ -9,7 +9,15 @@
 
 SET(SWIFTEST_BINDINGS _bindings)
 
-FIND_PROGRAM(CYTHON "cython")
+# Find the Cython executable, but don't look in the cmake root directory (due how cmake and cython are installed on the RCAC system)
+FIND_PROGRAM(CYTHON 
+            NAMES "cython" 
+            NO_CMAKE_PATH
+            NO_CMAKE_ENVIRONMENT_PATH
+            NO_CMAKE_SYSTEM_PATH
+            NO_CMAKE_FIND_ROOT_PATH
+            )
+MESSAGE(STATUS "Cython executable path: ${CYTHON}")
 ADD_CUSTOM_COMMAND(
   OUTPUT "${SWIFTEST_BINDINGS}.c"
   DEPENDS "${SWIFTEST_BINDINGS}.pyx"