Merge pull request #177 from mkoeppe/pyproject_toml

Add pyproject.toml, remove Cython from setup_requires, install_requires
sagemath · Sep 21, 2023 · 4b4d54a · 4b4d54a
2 parents d9350f7 + 63d401f
commit 4b4d54a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/.github/workflows/ci-sage.yml b/.github/workflows/ci-sage.yml
@@ -111,7 +111,7 @@ jobs:
         choco install make autoconf gcc-core gcc-g++ python3${{ matrix.python-version }}-devel --source cygwin
     - name: install dependencies
       run: |
-        C:\\tools\\cygwin\\bin\\bash -l -x -c 'export PATH=/usr/local/bin:/usr/bin && cd $(cygpath -u "$GITHUB_WORKSPACE") && python3.${{ matrix.python-version }} -m pip install setuptools cysignals Cython Sphinx flake8'
+        C:\\tools\\cygwin\\bin\\bash -l -x -c 'export PATH=/usr/local/bin:/usr/bin && cd $(cygpath -u "$GITHUB_WORKSPACE") && python3.${{ matrix.python-version }} -m pip install setuptools cysignals 'Cython<3' Sphinx flake8'
     - name: install
       run: |
         C:\\tools\\cygwin\\bin\\bash -l -x -c 'export PATH=/usr/local/bin:/usr/bin && cd $(cygpath -u "$GITHUB_WORKSPACE") && python3.${{ matrix.python-version }} setup.py build_ext -i'
@@ -241,7 +241,7 @@ jobs:
     - name: Install dependencies
       run: |
           python -m pip install --upgrade pip
-          pip install setuptools Cython Sphinx flake8
+          pip install setuptools 'Cython<3' Sphinx flake8
     - name: Freeze pip
       run: |
           pip freeze
@@ -354,7 +354,7 @@ jobs:
     - name: Install dependencies
       run: |
           python -m pip install --upgrade pip
-          pip install setuptools Cython Sphinx flake8
+          pip install setuptools 'Cython<3' Sphinx flake8
     - name: Freeze pip
       run: |
           pip freeze

diff --git a/pyproject.toml b/pyproject.toml
@@ -0,0 +1,2 @@
+[build-system]
+requires = ['setuptools', 'Cython>=0.28, <3']
diff --git a/setup.py b/setup.py
@@ -201,9 +201,6 @@ def run(self):
     long_description=README,
     long_description_content_type='text/x-rst',
     classifiers=classifiers,
-    install_requires=["Cython>=0.28"],
-    setup_requires=["Cython>=0.28"],
-
     ext_modules=extensions,
     packages=["cysignals"],
     package_dir={"": "src"},
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		[build-system]
		requires = ['setuptools', 'Cython>=0.28, <3']