Merge pull request #221 from aisk/build-dependencies

chore: add pyproject.toml and specify build dependencies
Thriftpy · Aug 7, 2023 · c8e5dd4 · c8e5dd4
2 parents da030d3 + 1b80073
commit c8e5dd4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 13 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -0,0 +1,2 @@
+[build-system]
+requires = ["setuptools", "cython>=0.28.4,<4"]
diff --git a/setup.py b/setup.py
@@ -31,7 +31,6 @@
     pass
 
 dev_requires = [
-    "cython>=0.28.4",
     "flake8>=2.5",
     "pytest>=2.8",
     "sphinx-rtd-theme>=0.1.9",
@@ -40,13 +39,6 @@
 ] + tornado_requires
 
 
-# cython detection
-try:
-    from Cython.Build import cythonize
-    CYTHON = True
-except ImportError:
-    CYTHON = False
-
 cmdclass = {}
 ext_modules = []
 
@@ -56,11 +48,10 @@
 
 # only build ext in CPython with UNIX platform
 if UNIX and not PYPY:
-    # rebuild .c files if cython available
-    if CYTHON:
-        cythonize("thriftpy2/transport/cybase.pyx")
-        cythonize("thriftpy2/transport/**/*.pyx")
-        cythonize("thriftpy2/protocol/cybin/cybin.pyx")
+    from Cython.Build import cythonize
+    cythonize("thriftpy2/transport/cybase.pyx")
+    cythonize("thriftpy2/transport/**/*.pyx")
+    cythonize("thriftpy2/protocol/cybin/cybin.pyx")
 
     ext_modules.append(Extension("thriftpy2.transport.cybase",
                                  ["thriftpy2/transport/cybase.c"]))
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		[build-system]
		requires = ["setuptools", "cython>=0.28.4,<4"]