search

IndigoResearch / textteaser

Official version of TextTeaser.
MIT License
621 stars 143 forks source link

added setup.py #1

Open JonnoFTW opened 9 years ago

JonnoFTW commented 9 years ago

Added a setup.py to make this package easier to use. Requires setuptools.

MojoJolo commented 9 years ago

@JonnoFTW sorry I just saw this pull request. I wonder why it didn't alert me in my email. Will try this out and merge if all is good.

neurosnap commented 9 years ago

https://pypi.python.org/pypi/textteaser/0.3

I already see "textteaser" python package, if you want to upload this package to pypi -- which is recommended -- then you'll prolly want to go with a different name.

MojoJolo commented 9 years ago

Thanks. It looks like he's using a previous version of TextTeaser that doesn't work today. I wonder if it is possible to replace it.

neurosnap commented 9 years ago

I would recommend something like 'textteaserpy' or even 'txtteaser' if you want to change the name for pypi.

I would also recommend manually writing the install_requires since there is only one dependency and reading the file line-by-line has its complications: empty lines, invalid package names, and also doesn't allow the ability to download packages from github.

See this http://stackoverflow.com/a/16624700/1713216 post for more information on reading the requirements.txt for setup.py

JonnoFTW commented 9 years ago

For all this to happen, the DataTeaser team would need to come up with an appropriate package name, (I just wrote the setup.py file).