Migrate from setuptools to scikit-build-core (#121)

- Replace setuptools build backend with scikit-build-core in pyproject.toml
- Add CMakeLists.txt for building Cython extension with platform-specific RPATH

Author: urschrei

CMakeLists.txt

@@ -0,0 +1,75 @@
+cmake_minimum_required(VERSION 3.15...3.27)
+project(${SKBUILD_PROJECT_NAME} LANGUAGES C)
+
+# Find Python
+find_package(
+  Python
+  COMPONENTS Interpreter Development.Module NumPy
+  REQUIRED)
+
+# 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/simplification/cutil.pyx"
+  COMMAND Python::Interpreter -m cython
+          "${CMAKE_CURRENT_SOURCE_DIR}/src/simplification/cutil.pyx"
+          --output-file cutil.c
+          -I "${CMAKE_CURRENT_SOURCE_DIR}/src/simplification"
+  DEPENDS src/simplification/cutil.pyx src/simplification/rdp_p.pxd
+  VERBATIM)
+
+# Create the extension module
+python_add_library(cutil MODULE cutil.c WITH_SOABI)
+
+# Set include directories
+target_include_directories(cutil PRIVATE
+  ${CMAKE_CURRENT_SOURCE_DIR}/src/simplification
+  ${Python_NumPy_INCLUDE_DIRS}
+)
+
+# Link against the rdp library
+# The library is pre-built and located in src/simplification/
+find_library(RDP_LIBRARY
+  NAMES rdp librdp
+  PATHS ${CMAKE_CURRENT_SOURCE_DIR}/src/simplification
+  NO_DEFAULT_PATH
+  REQUIRED
+)
+message(STATUS "Found RDP library: ${RDP_LIBRARY}")
+
+target_link_libraries(cutil PRIVATE ${RDP_LIBRARY})
+
+# Set RPATH for different platforms
+if(APPLE)
+  set_target_properties(cutil PROPERTIES
+    INSTALL_RPATH "@loader_path"
+    BUILD_WITH_INSTALL_RPATH TRUE
+  )
+elseif(UNIX)
+  set_target_properties(cutil PROPERTIES
+    INSTALL_RPATH "$ORIGIN"
+    BUILD_WITH_INSTALL_RPATH TRUE
+  )
+endif()
+
+# Install the extension module
+install(TARGETS cutil DESTINATION simplification)
+
+# Install the shared library
+if(APPLE)
+  set(RDP_LIB_NAME "librdp.dylib")
+elseif(UNIX)
+  set(RDP_LIB_NAME "librdp.so")
+elseif(WIN32)
+  set(RDP_LIB_NAME "rdp.dll")
+endif()
+
+install(FILES ${CMAKE_CURRENT_SOURCE_DIR}/src/simplification/${RDP_LIB_NAME}
+        DESTINATION simplification)
+
+# Install header file
+install(FILES ${CMAKE_CURRENT_SOURCE_DIR}/src/simplification/header.h
+        DESTINATION simplification)

README.md

@@ -11,9 +11,14 @@ Simplify a LineString using the [Ramer–Douglas–Peucker](https://en.wikipedia
 `conda install conda-forge::simplification`
 
 ### Installing for local development
-1. Ensure you have a copy of `librdp` from https://github.com/urschrei/rdp/releases, and it's in the `src/simplification` subdir
-2. run `pip install -e .[test] --use-pep517`
+1. Ensure you have a copy of `librdp` and `header.h` from https://github.com/urschrei/rdp/releases, and it's in the `src/simplification` subdir
+2. run `uv sync --dev`
 3. run `pytest .`
+4. If you make changes, you must rebuild the extension: `uv sync --reinstall`
+
+## Building SDist and Wheels
+1. Ensure that `librdp` and header are present, as above
+2. Run `uv build --sdist --wheel`
 
 ### Supported Python Versions
 Simplification supports all [_currently_ supported Python versions](https://devguide.python.org/versions/).

pyproject.toml

@@ -1,7 +1,8 @@
 [project]
 name = "simplification"
-dynamic = ["version", "readme"]
+version = "0.7.15"
 description = "Fast linestring simplification using RDP or Visvalingam-Whyatt and a Rust binary"
+readme = "README.md"
 requires-python = ">=3.10"
 dependencies = [
     "numpy >= 2.0.0",
@@ -29,37 +30,28 @@ classifiers = [
 Repository = "https://github.com/urschrei/simplification"
 Tracker = "https://github.com/urschrei/simplification/issues"
 
-[project.optional-dependencies]
-test = ["pytest >= 7.4.2"]
-
-[tool.setuptools.packages.find]
-where = ["src"]
-
-[tool.setuptools.package-data]
-simplification = ["*.dylib", "*.dll", "*.so", "*.h"]
+[dependency-groups]
+dev = [
+    "pytest>=8.4.1",
+]
 
-[tool.setuptools.dynamic]
-readme = {file = ["README.md", "LICENSE.md"], content-type = "text/markdown"}
+[tool.scikit-build]
+minimum-version = "build-system.requires"
+wheel.packages = ["src/simplification"]
 
 [build-system]
-build-backend = "setuptools.build_meta"
 requires = [
-    "setuptools >= 45",
-    "setuptools-scm[toml] >= 6.2",
+    "scikit-build-core>=0.10",
     "numpy >= 2.0.0",
-    "cython >= 3.0.0",
-    "wheel >= 0.29.0"
+    "cython >= 3.1.0",
 ]
+build-backend = "scikit_build_core.build"
 
 [tool.pytest.ini_options]
-minversion = "6.2.2"
+minversion = "8.4.1"
 addopts = [
     "--import-mode=importlib",
 ]
 testpaths = [
     "tests",
 ]
-
-[tool.setuptools_scm]
-write_to = "src/_version.py"
-