Merge pull request benoitc#88 from adamnovak/require-cython-when-used

benoitc · web-flow · commit 83ef694621e3 · 2020-04-05T17:08:17.000+02:00
Require a recent Cython when building C code
diff --git a/pyproject.toml b/pyproject.toml
@@ -0,0 +1,8 @@
+# This file is renamed to "setup.toml.dev" in release tarballs so that
+# Pip won't try to use it.
+
+[build-system]
+# In addition to setuptools and wheel, we need modern Cython, because we are
+# not a packaged release.
+requires = ["setuptools", "wheel", "cython>=0.28.3"]
+
diff --git a/pyproject.toml.release b/pyproject.toml.release
@@ -0,0 +1,7 @@
+# This file is renamed to "setup.toml" in release tarballs so that
+# Pip will use it.
+
+[build-system]
+# Release packages only need the standard setuptools and wheel.
+requires = ["setuptools", "wheel"]
+
diff --git a/requirements_dev.txt b/requirements_dev.txt
@@ -1,2 +1,2 @@
 pytest
-cython
+cython>=0.28.3
diff --git a/setup.py b/setup.py
@@ -82,12 +82,16 @@ def run(self):
         if os.path.exists('Makefile'):
             make()
             os.rename('Makefile', 'Makefile.ext')
+            os.rename('pyproject.toml', 'pyproject.toml.dev')
+            os.rename('pyproject.toml.release', 'pyproject.toml')
             renamed = True
         try:
             return _sdist.run(self)
         finally:
             if renamed:
                 os.rename('Makefile.ext', 'Makefile')
+                os.rename('pyproject.toml', 'pyproject.toml.release')
+                os.rename('pyproject.toml.dev', 'pyproject.toml')
 
 class BuildFailed(Exception):
     pass
