-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Not python 3 compatible. #2
Comments
I've started reading through some of the code and see other Python 3 inconsistencies, such as https://github.com/crorella/hots-parser/blob/master/main.py#L76 So really this ticket should be expanded to just be: Be Python 3 compatible. |
Yes you are right, most of the prints are not Python 3 compatible. I would love to do it but for now there are other high-pri tasks I'm working on. If you would like to make it python 3 friendly please go ahead and I will help with the review. |
I've run into a Python 2 dependency while trying
pip3 install -r requirements.txt
inside a Python 3 virtual environment.Upon further investigation, I see that according to https://pypi.python.org/pypi/psycopg2ct, this package is only tested on Python 2.6+.
I would recommend either removing this package and switching to something Python 3 compatible, or at least putting it in the readme you must use Python 2.
I was able to
pip install -r requirements.txt
in a Python 2 virtual environment no problem though. For now that is the work around.Edit: This was tested on Ubuntu 17.04, Python 3.5.3, Python 2.7.13, and GCC 6.3.0 20170118.
The text was updated successfully, but these errors were encountered: