Switch to cython-cmake util for scikit-build (#108)

Author: urschrei

CMakeLists.txt

@@ -1,40 +1,40 @@
-cmake_minimum_required(VERSION 3.15)
+cmake_minimum_required(VERSION 3.21)
 project(${SKBUILD_PROJECT_NAME} LANGUAGES C)
 
+# Define source directory
+set(convertbng_SRC_DIR "${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng")
+
 # Find Python
 find_package(
   Python
   COMPONENTS Interpreter Development.Module NumPy
   REQUIRED)
+include(UseCython)
 
 # Get NumPy include directory
 message(STATUS "NumPy include directory: ${Python_NumPy_INCLUDE_DIRS}")
 
-# Cythonize the .pyx file to .c
-add_custom_command(
-  OUTPUT cutil.c
-  COMMENT "Cythonizing ${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng/cutil.pyx"
-  COMMAND Python::Interpreter -m cython
-          "${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng/cutil.pyx"
-          --output-file cutil.c
-          -I "${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng"
-  DEPENDS src/convertbng/cutil.pyx src/convertbng/convertbng_p.pxd
-  VERBATIM)
+cython_transpile(
+  "${convertbng_SRC_DIR}/cutil.pyx"
+  LANGUAGE C
+  OUTPUT_VARIABLE cutil
+  CYTHON_ARGS -I "${convertbng_SRC_DIR}"
+)
 
 # Create the extension module
-python_add_library(cutil MODULE cutil.c WITH_SOABI)
+python_add_library(cutil MODULE "${cutil}" WITH_SOABI)
 
 # Set include directories
 target_include_directories(cutil PRIVATE
-  ${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng
+  ${convertbng_SRC_DIR}
   ${Python_NumPy_INCLUDE_DIRS}
 )
 
 # Link against the Rust library
 # The library is pre-built and located in src/convertbng/
 find_library(LONLAT_BNG_LIBRARY
   NAMES lonlat_bng liblonlat_bng
-  PATHS ${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng
+  PATHS ${convertbng_SRC_DIR}
   NO_DEFAULT_PATH
   REQUIRED
 )
@@ -60,16 +60,16 @@ install(TARGETS cutil DESTINATION convertbng)
 
 # Install the shared library
 if(APPLE)
-  set(RDP_LIB_NAME "liblonlat_bng.dylib")
+  set(LONLAT_BNG_LIB_NAME "liblonlat_bng.dylib")
 elseif(UNIX)
-  set(RDP_LIB_NAME "liblonlat_bng.so")
+  set(LONLAT_BNG_LIB_NAME "liblonlat_bng.so")
 elseif(WIN32)
-  set(RDP_LIB_NAME "lonlat_bng.dll")
+  set(LONLAT_BNG_LIB_NAME "lonlat_bng.dll")
 endif()
 
-install(FILES ${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng/${RDP_LIB_NAME}
+install(FILES ${convertbng_SRC_DIR}/${LONLAT_BNG_LIB_NAME}
         DESTINATION convertbng)
 
 # Install header file
-install(FILES ${CMAKE_CURRENT_SOURCE_DIR}/src/convertbng/header.h
+install(FILES ${convertbng_SRC_DIR}/header.h
         DESTINATION convertbng)

pyproject.toml

@@ -44,6 +44,7 @@ requires = [
     "scikit-build-core>=0.10",
     "numpy >= 2.0.0",
     "cython >= 3.1.0",
+    "cython-cmake"
 ]
 build-backend = "scikit_build_core.build"