Merge pull request #37 from jmalkin/build_wheels

Build wheels

Author: c-dickens

.github/workflows/build_wheels.yml

@@ -5,6 +5,7 @@ on:
 
 env:
   BUILD_TYPE: Release
+  MIN_CIBUILDWHEEL_VERSION: 2.16.2
 
 jobs:
   build_sdist:
@@ -82,18 +83,12 @@ jobs:
           platforms: arm64
 
       - name: Install Python dependencies
-        run: python -m pip install cibuildwheel==2.16.2
+        run: python -m pip install cibuildwheel~=${{ env.MIN_CIBUILDWHEEL_VERSION }}
 
       - name: Build wheels
         run: python -m cibuildwheel --output-dir dist
         env:
-          CIBW_ARCHS_LINUX: "auto aarch64"
-          CIBW_ARCHS_MACOS: "x86_64 arm64"
-          CIBW_ENVIRONMENT_MACOS: CMAKE_OSX_ARCHITECTURES=${{ matrix.config.cibw-arch == 'macosx_x86_64' && 'x86_64' || matrix.config.cibw-arch == 'macosx_arm64' && 'arm64' || '' }}
           CIBW_BUILD: "*-${{ matrix.config.cibw-arch }}"
-          CIBW_BEFORE_BUILD_LINUX: "yum remove -y cmake"
-          CIBW_BEFORE_BUILD: "python -m pip install cmake>=3.18"
-          CIBW_SKIP: "*-win32 pp*-aarch64 pp*-macosx"
 
       - uses: actions/upload-artifact@v3
         with:

CMakeLists.txt

@@ -65,7 +65,7 @@ find_package(Python COMPONENTS Interpreter Development NumPy REQUIRED)
 endif()
 
 if(Python_VERSION VERSION_LESS "3.8")
-  message(FATAL_ERROR "DataSketches requires at least Python3.")
+  message(FATAL_ERROR "DataSketches requires at least Python3.8")
 endif()
 
 execute_process(

pyproject.toml

@@ -19,6 +19,23 @@
 requires = ["wheel",
             "setuptools >= 30.3.0",
             "cmake >= 3.16",
-            "numpy",
+            "numpy < 2.0",
             "nanobind >= 1.6"]
 build-backend = "setuptools.build_meta"
+
+[tool.cibuildwheel]
+build-verbosity = 0  # options: 1, 2, or 3 
+skip = ["cp36-*", "cp37-*", "pp*", "*-win32"]
+
+[tool.cibuildwheel.windows]
+archs = ["auto64"]
+
+[tool.cibuildwheel.linux]
+archs = ["auto", "aarch64"]
+before-build = "yum remove -y cmake"
+
+[tool.cibuildwheel.macos]
+archs = ["x86_64", "arm64"]
+
+# Minimum version for proper C++17 support on MacOS
+environment = { MACOSX_DEPLOYMENT_TARGET = "10.14" }