revelc / pyaccumulo

Python Client Library for Apache Accumulo
Apache License 2.0
26 stars 23 forks source link

ez_setup.py is very old #23

Closed tristeng closed 9 years ago

tristeng commented 9 years ago

It installs a very old version of setup tools, and I think that setup.py should probably attempt to load setup tools functions before trying to import from ez_setup.py. In my virtual environment, it was installing the old version of setup tools over top of a much newer version of setup tools, and I was seeing some errors like "Failed building wheel..." for certain packages I was trying to install.

Consider replacing this:

try:
    from ez_setup import use_setuptools
    use_setuptools()
except ImportError:
    pass

try:
    from setuptools import setup
except ImportError:
    from distutils.core import setup

with this:

try:
    from setuptools import setup, find_packages
except ImportError:
    from ez_setup import use_setuptools
    use_setuptools()
    from setuptools import setup, find_packages

I'm not a python setup.py expert, but the above change in my forked repository is working for me.

johnrfrank commented 9 years ago

Thanks for pointing that out. I've fixed it in 1.5.0.9