add cython to setup_requires if needed

zeromq · Nov 25, 2020 · cc0538c · cc0538c
1 parent 922197e
commit cc0538c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -76,6 +76,7 @@ jobs:
         if: "! startsWith(matrix.python, 'pypy')"
         run: |
           pip install cython
+          python setup.py cython
 
       - name: remove tornado
         if: matrix.tornado == 'none'

diff --git a/setup.py b/setup.py
@@ -1366,13 +1366,17 @@ def find_packages():
     zip_safe=False,
     python_requires=">=3.6",
     install_requires=[
-        "py; implementation_name === 'pypy'",
-        "cffi; implementation_name === 'pypy'",
+        "py; implementation_name == 'pypy'",
+        "cffi; implementation_name == 'pypy'",
     ],
     setup_requires=[
-        "cffi; implementation_name === 'pypy'",
+        "cffi; implementation_name == 'pypy'",
     ],
 )
+if not os.path.exists(os.path.join("zmq", "backend", "cython", "socket.c")):
+    setup_args["setup_requires"].append(
+        f"cython>={min_cython_version}; implementation_name == 'cpython'"
+    )
 
 
 setup(**setup_args)