Skip to content

Commit

Permalink
Make sure install_requires always contains numpy.
Browse files Browse the repository at this point in the history
install_requires does not need to contain Cython
  • Loading branch information
grlee77 committed Mar 12, 2018
1 parent f11944a commit 61427a5
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -742,10 +742,11 @@ def setup_package():

# Figure out whether to add ``*_requires = ['numpy']``.
build_requires = []
numpy_requirement = 'numpy>=1.6, <2.0'
try:
import numpy
except ImportError:
build_requires = ['numpy>=1.6, <2.0']
build_requires = [numpy_requirement]

# we require cython because we need to know which part of the wrapper
# to build to avoid missing symbols at runtime. But if this script is
Expand All @@ -756,8 +757,7 @@ def setup_package():
except ImportError:
build_requires.append('cython>=0.23, <1.0')

install_requires = []
install_requires.extend(build_requires)
install_requires = [numpy_requirement]

setup_args = {
'name': 'pyFFTW',
Expand Down

0 comments on commit 61427a5

Please sign in to comment.