bbangert / retools

Redis Tools
MIT License
138 stars 41 forks source link

Removed setup_requires nose #27

Closed heynemann closed 9 years ago

heynemann commented 10 years ago

I removed it because it becomes a dependency whenever installing retools in production. This would not be an issue if there was not a bug in setuptools.

If there's any packages in setup_requires, setuptools downloads it without respecting any proxy configuration. Since many production servers block outbound connections, it's desirable to remove this attribute.

heynemann commented 10 years ago

Any news on this issue?

msabramo commented 9 years ago

:+1: I often find setup_requires to be problematic. The problem that has hit me a fair amount: https://bitbucket.org/pypa/setuptools/issue/209/setup_requires-and-install_requires-dont

Usually I just put dev and testing dependencies in a dev_requirements.txt or test_requirements.txt.