Move to scikit-build-core

Author: isuruf

.gitignore

@@ -20,6 +20,7 @@ cython_test.cpp
 symengine/lib/symengine_wrapper.cpp
 symengine/lib/symengine_wrapper.pyx
 symengine/lib/symengine_wrapper.pxd
+symengine/_version.py
 
 # Config Files
 symengine/lib/config.pxi
@@ -33,6 +34,12 @@ MANIFEST
 dist/
 .*cache/
 symengine.egg-info/
+.cmake/
+.ninja_deps
+.ninja_log
+.skbuild-info.json
+CMakeInit.txt
+symengine/lib/symengine_wrapper.cpp.dep
 
 # Temp files
 *~

CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 2.8.12...4.0.0)
+cmake_minimum_required(VERSION 3.21...4.0.0)
 
 if (POLICY CMP0057)
   cmake_policy(SET CMP0057 NEW) # needed for llvm >= 16
@@ -7,7 +7,7 @@ if (POLICY CMP0074)
   cmake_policy(SET CMP0074 NEW)  # allow user to set *_ROOT variables
 endif()
 
-project(python_wrapper)
+project(${SKBUILD_PROJECT_NAME} LANGUAGES C CXX)
 
 set(CMAKE_PREFIX_PATH ${SymEngine_DIR} ${CMAKE_PREFIX_PATH})
 
@@ -21,13 +21,28 @@ set(CMAKE_BUILD_TYPE ${SYMENGINE_BUILD_TYPE})
 set(CMAKE_CXX_FLAGS_RELEASE ${SYMENGINE_CXX_FLAGS_RELEASE})
 set(CMAKE_CXX_FLAGS_DEBUG ${SYMENGINE_CXX_FLAGS_DEBUG})
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${SYMENGINE_CXX_FLAGS}")
-include_directories(${SYMENGINE_INCLUDE_DIRS})
 
-set(WITH_PY_LIMITED_API OFF CACHE STRING "Use CPython's limited API")
-set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_CURRENT_SOURCE_DIR}/cmake/")
-find_package(Python REQUIRED)
-find_package(Cython REQUIRED)
-include_directories(${PYTHON_INCLUDE_PATH})
+find_package(
+  Python
+  COMPONENTS
+    Interpreter
+    Development.Module
+    ${SKBUILD_SABI_COMPONENT}
+  REQUIRED
+)
+
+if ("${SKBUILD_SABI_VERSION}" STREQUAL "")
+  set(CYTHON_MIN "0.29")
+else()
+  set(CYTHON_MIN "3.1")
+endif()
+
+find_package(Cython ${CYTHON_MIN} MODULE REQUIRED)
+include(UseCython)
+
+if ("${CYTHON_VERSION}" VERSION_LESS "${CYTHON_MIN}")
+  message(FATAL_ERROR "Cython version found: ${CYTHON_VERSION}. Minimum required: ${CYTHON_MIN}")
+endif()
 
 if (MINGW AND ("${CMAKE_SIZEOF_VOID_P}" STREQUAL "8"))
     set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -DMS_WIN64")
@@ -56,7 +71,6 @@ foreach (PKG MPC MPFR PIRANHA FLINT LLVM)
         set(HAVE_SYMENGINE_${PKG} False)
     endif()
 endforeach()
-option(SYMENGINE_INSTALL_PY_FILES "Install python files" ON)
 
 message("CMAKE_SYSTEM_PROCESSOR              : ${CMAKE_SYSTEM_PROCESSOR}")
 message("CMAKE_BUILD_TYPE                    : ${CMAKE_BUILD_TYPE}")
@@ -69,9 +83,5 @@ message("HAVE_SYMENGINE_PIRANHA              : ${HAVE_SYMENGINE_PIRANHA}")
 message("HAVE_SYMENGINE_FLINT                : ${HAVE_SYMENGINE_FLINT}")
 message("HAVE_SYMENGINE_LLVM                 : ${HAVE_SYMENGINE_LLVM}")
 message("HAVE_SYMENGINE_LLVM_LONG_DOUBLE     : ${HAVE_SYMENGINE_LLVM_LONG_DOUBLE}")
-message("SYMENGINE_COPY_EXTENSION            : ${SYMENGINE_COPY_EXTENSION}")
-
-message("Copying source of python wrappers into: ${CMAKE_CURRENT_BINARY_DIR}")
-file(COPY symengine/ DESTINATION ${CMAKE_CURRENT_BINARY_DIR}/symengine)
 
 add_subdirectory(symengine)

README.md

@@ -27,10 +27,14 @@ conda install python-symengine -c conda-forge
 Install prerequisites.
 
 ```bash
-CMake       >= 2.8.12
-Python3     >= 3.8
-Cython      >= 0.29.24
-SymEngine   >= 0.7.0
+CMake       >= 3.21
+Python3     >= 3.9
+SymEngine   >= 0.14.0
+pip
+setuptools_scm           # will be automatically downloaded by pip
+scikit-build-core        # will be automatically downloaded by pip
+cython      >= 0.29.24   # will be automatically downloaded by pip
+cython-cmake             # will be automatically downloaded by pip
 ```
 
 For **SymEngine**, only a specific commit/tag (see `symengine_version.txt`) is
@@ -40,25 +44,22 @@ changes in **SymEngine**.
 Python wrappers can be installed by,
 
 ```bash
-python setup.py install
+pip install . -vv
 ```
 
-Additional options to `setup.py` are:
-
+If you are building SymEngine for development, you should implement
+the dependencies automatically installed by pip and do,
 ```bash
-python setup.py install build_ext
-    --symengine-dir=/path/to/symengine/install/dir          # Path to SymEngine install directory or build directory
-    --compiler=mingw32|msvc|cygwin                          # Select the compiler for Windows
-    --generator=cmake-generator                             # CMake Generator
-    --build-type=Release|Debug                              # Set build-type for multi-configuration generators like MSVC
-    --define="var1=value1;var2=value2"                      # Give options to CMake
-    --inplace                                               # Build the extension in source tree
+pip install -e . -vv --no-build-isolation
 ```
 
-Standard options to `setup.py` like `--user`, `--prefix` can be used to
-configure install location.  NumPy is used if found by default, if you wish
-to make your choice of NumPy use explicit: then add
-e.g. `WITH_NUMPY=False` to `--define`.
+A few additional options to scikit-build-core are mentioned here:
+
+```bash
+pip install -e . -vv --no-build-isolation
+    -Ccmake.build-type=Release|Debug                                # Set build-type for multi-configuration generators like MSVC
+    -Ccmake.define.SymEngine_DIR=/path/to/symengine/install/dir     # Path to SymEngine install directory or build directory
+```
 
 ### Notes on Dependencies
 

bin/install_travis.sh

@@ -2,7 +2,7 @@
 
 # symengine's bin/install_travis.sh will install miniconda
 
-export conda_pkgs="python=${PYTHON_VERSION} pip pytest setuptools gmp mpfr"
+export conda_pkgs="python=${PYTHON_VERSION} pip pytest python-build scikit-build-core ninja cython-cmake cython setuptools-scm gmp mpfr"
 
 if [[ "${WITH_NUMPY}" != "no" ]]; then
     export conda_pkgs="${conda_pkgs} numpy";
@@ -31,7 +31,7 @@ if [[ "${WITH_SAGE}" == "yes" ]]; then
     export conda_pkgs="${conda_pkgs} sage=8.1";
 fi
 
-conda install -q ${conda_pkgs} "cython>=0.29.24"
+conda install -q ${conda_pkgs}
 
 if [[ "${WITH_SYMPY}" != "no" ]]; then
     pip install sympy;

bin/test_travis.sh

@@ -5,16 +5,17 @@ set -e
 # Echo each command
 set -x
 
-python setup.py sdist
+python -m build . --sdist
 mkdir dist-extract
 cd dist-extract
 tar -xvf ../dist/symengine-*.tar.gz
 cd symengine-*
 
-# Build inplace so that nosetests can be run inside source directory
-python3 setup.py install build_ext --inplace --symengine-dir=$our_install_dir
-
-if [[ "${SYMENGINE_PY_LIMITED_API:-}" != "" ]]; then
+# Build inplace
+if [[ "${SYMENGINE_PY_LIMITED_API:-}" == "" ]]; then
+  python3 -m pip install -e . -vv -Ccmake.define.SymEngine_DIR=$our_install_dir
+else
+  python3 -m pip install -e . -vv -Ccmake.define.SymEngine_DIR=$our_install_dir -Cwheel.py-api="cp${SYMENGINE_PY_LIMITED_API/./}"
   python3 -m abi3audit --assume-minimum-abi3 ${SYMENGINE_PY_LIMITED_API} symengine/lib/symengine_wrapper.abi3.so -v
 fi