Correctly specify dependencies

Author: jdemeyer

.travis.yml

@@ -30,7 +30,6 @@ before_install:
   - ccache -M 256M && ccache -s
 install:
   - bash -x .travis-install-pari.sh
-  - pip install --build=/tmp/pip --verbose -r requirements.txt
   # CFLAGS:
   # -O1 to speed up compilation (compared to -O2 or -O3)
   # -fno-strict-aliasing because that is the default in Python
@@ -42,6 +41,7 @@ install:
 # command to run tests
 script:
   - make check
+  - pip install sphinx
   - (cd docs && make html)
 cache:
   directories:

README.rst

@@ -28,7 +28,7 @@ Requirements:
 - PARI/GP >= 2.9.4 (header files and library)
 - Python 2.7 or Python >= 3.4
 - pip
-- `cysignals <https://pypi.python.org/pypi/cysignals/>`_
+- `cysignals <https://pypi.python.org/pypi/cysignals/>`_ >= 1.7
 - Cython >= 0.28
 
 Install cypari2 via the Python Package Index (PyPI) via

setup.py

@@ -30,22 +30,6 @@
 # Adapted from Cython's new_build_ext
 class build_ext(_build_ext):
     def finalize_options(self):
-        # Check dependencies
-        try:
-            from Cython.Build.Dependencies import cythonize
-        except ImportError as E:
-            sys.stderr.write("Error: {0}\n".format(E))
-            sys.stderr.write("The installation of cypari2 requires Cython\n")
-            sys.exit(1)
-
-        try:
-            # We need the header files for cysignals at compile-time
-            import cysignals
-        except ImportError as E:
-            sys.stderr.write("Error: {0}\n".format(E))
-            sys.stderr.write("The installation of cypari2 requires cysignals\n")
-            sys.exit(1)
-
         # Generate auto-generated sources from pari.desc
         rebuild()
 
@@ -56,11 +40,16 @@ def finalize_options(self):
             "language_level": 2,
         }
 
+        _build_ext.finalize_options(self)
+
+    def run(self):
+        # Run Cython
+        from Cython.Build.Dependencies import cythonize
         self.distribution.ext_modules[:] = cythonize(
             self.distribution.ext_modules,
             compiler_directives=self.directives)
 
-        _build_ext.finalize_options(self)
+        _build_ext.run(self)
 
 
 class no_egg(_bdist_egg):
@@ -79,6 +68,8 @@ def run(self):
 setup(
     name='cypari2',
     version=VERSION,
+    setup_requires=['Cython>=0.28'],
+    install_requires=['cysignals>=1.7'],
     description="A Python interface to the number theory library PARI/GP",
     long_description=README,
     url='https://github.com/defeo/cypari2',