jquast / x84

A python telnet/ssh server for modern terminals. In spirit of classic software such as ami/x, teleguard, renegade, iniquity.

Home Page:http://x84.readthedocs.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

requirements.txt is missing feedparser and html2text entries (needed for hackernews.py)

opened this issue · comments

I've just done some testing of the new 2.0 release and noticed that hackernews.py threw an exception due to missing 'feedparser' and 'html2text' Python modules. After installing these two manually using 'pip' everything works just fine. In order to easily set up the environment, I'd request to have those missing modules added to requirements.txt

Why/How are you using requirements.txt? do you just decide for yourself to use it? Why aren't you installing x/84 as a package, either from pip/pypi, or from "setup.py develop -U" ? In which case, these are described in setup.py.

If anything, the requirements.txt file should be deleted.

Well, I just did a git pull and pulled all recent changes from upstream. I do not use virtualenv on my development system and just pull all changes from my cloned repo as well as from upstream. so it all is located under $HOME/src/x84. maybe - and I am quite sure this is the case - I do misunderstand how to proplerly set up my dev environment. I did follow the "developers" section in the docs though. However, if this is not really a problem due to me not getting things right, feel free to close this issue.

Ah, and thanks for merging the recent lightbar_main.py changes. There's more to come.

deleting requirements*.txt entirely for next release