machinalis / iepy

Information Extraction in Python
BSD 3-Clause "New" or "Revised" License
906 stars 186 forks source link

Raise warning if installing with python 2 #50

Closed rafacarrascosa closed 9 years ago

rafacarrascosa commented 9 years ago

Enhancement proposal by @makmanalp thanks for reporting!

"While installing, it took me a minute to realize that it was python 3 only, which is not a big deal, but it might help to have something in setup.py that errors out and gives you a message if you try to do that - or maybe there's a way in pypi to disallow from installing on python 2?"

jmansilla commented 9 years ago

Couldn't find how to do this (pypi provides a python-2-only label, but not a python-3-only), and putting something on setup.py doesn't seem to solve anything, because the python2 finds syntax errors before even trying.

https://pypi.python.org/pypi?%3Aaction=list_classifiers

rafacarrascosa commented 9 years ago

Great! Thanks Javi, I reported this to pypi:

https://sourceforge.net/p/pypi/support-requests/436/

Issue report snowball! Yaaay!

rafacarrascosa commented 9 years ago

Monsieur Mansilla your wishes are my commands, new trove classifier added into pypi :D

mgaitan commented 9 years ago

On Fri, Nov 14, 2014 at 3:13 AM, rafacarrascosa notifications@github.com wrote:

Monsieur Mansilla your wishes are my commands, new trove classifier added into pypi :D

Made :: by :: Ñoños :: Only

is it available?

;-)

jmansilla commented 9 years ago

Buenísimo.

Ni idea si eso alcanzara para el request inicial, intentemos.

Javier Mansilla.

jmansilla commented 9 years ago

It's on develop now.

Hopefully a future version of PIP will be able to do something with packages having that label.