Skip to content

Commit

Permalink
BLD: since we already use setuptools, let's remove the optional logic…
Browse files Browse the repository at this point in the history
… in setup.py (GH18113).
  • Loading branch information
kchomski committed Nov 7, 2017
1 parent 537e880 commit 2bd0ffc
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 26 deletions.
2 changes: 1 addition & 1 deletion doc/source/whatsnew/v0.22.0.txt
Original file line number Diff line number Diff line change
Expand Up @@ -157,6 +157,6 @@ Categorical
Other
^^^^^

-
- BLD: since we already use setuptools, let's remove the optional logic in setup.py (:issue:`18113`)
-
-
28 changes: 3 additions & 25 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,11 @@
import os
from os.path import join as pjoin

import pkg_resources
import sys
import shutil
from distutils.version import LooseVersion
from setuptools import setup, Command

# versioning
import versioneer
Expand All @@ -38,14 +40,6 @@ def is_platform_mac():
except ImportError:
_CYTHON_INSTALLED = False

try:
import pkg_resources
from setuptools import setup, Command
_have_setuptools = True
except ImportError:
# no setuptools installed
from distutils.core import setup, Command
_have_setuptools = False

setuptools_kwargs = {}
min_numpy_ver = '1.9.0'
Expand All @@ -56,10 +50,6 @@ def is_platform_mac():
'pytz >= 2011k',
'numpy >= %s' % min_numpy_ver],
'setup_requires': ['numpy >= %s' % min_numpy_ver]}
if not _have_setuptools:
sys.exit("need setuptools/distribute for Py3k"
"\n$ pip install distribute")

else:
setuptools_kwargs = {
'install_requires': ['python-dateutil',
Expand All @@ -69,16 +59,6 @@ def is_platform_mac():
'zip_safe': False,
}

if not _have_setuptools:
try:
import numpy # noqa:F401
import dateutil # noqa:F401
setuptools_kwargs = {}
except ImportError:
sys.exit("install requires: 'python-dateutil < 2','numpy'."
" use pip or easy_install."
"\n $ pip install 'python-dateutil < 2' 'numpy'")

from distutils.extension import Extension # noqa:E402
from distutils.command.build import build # noqa:E402
from distutils.command.build_ext import build_ext as _build_ext # noqa:E402
Expand Down Expand Up @@ -696,9 +676,7 @@ def pxd(name):
sources=['pandas/util/move.c'])
extensions.append(_move_ext)


if _have_setuptools:
setuptools_kwargs["test_suite"] = "nose.collector"
setuptools_kwargs["test_suite"] = "nose.collector"

# The build cache system does string matching below this point.
# if you change something, be careful.
Expand Down

0 comments on commit 2bd0ffc

Please sign in to comment.