Skip to content
This repository has been archived by the owner on Jan 4, 2021. It is now read-only.

Syntax error when python -m pythorrent -h is run #3

Open
tejt99 opened this issue Aug 2, 2019 · 5 comments
Open

Syntax error when python -m pythorrent -h is run #3

tejt99 opened this issue Aug 2, 2019 · 5 comments

Comments

@tejt99
Copy link
Contributor

tejt99 commented Aug 2, 2019

Hi I am just fiddling around with this and when I run python -m pythorrent -h I get a large stream of errors:

Traceback (most recent call last):
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/Users/theoturner/Downloads/python-pythorrent/pythorrent/main.py", line 16, in
from .torrent import Torrent
File "pythorrent/torrent.py", line 26, in
from .peer_stores import store_from_url
File "pythorrent/peer_stores.py", line 57
class Tracker(object)
^
SyntaxError: invalid syntax

I am not sure what this means. At all, whatsoever.
I am using mac os high sierra.
Thanks in advance

@yamatt
Copy link
Owner

yamatt commented Aug 2, 2019

hi @tejt99. I'm very appreciative of you reporting this. Unfortunately I'm not in a position to support this project any more, but I am glad people are interested in it.

The issue with the line 57 you are seeing are due to a : missing from the end of it.

I would very much appreciate it if you were able to provide a Pull Request for the change and any other issues you may find so that others may benefit from it.

I hope you get some value from it.

@tejt99
Copy link
Contributor Author

tejt99 commented Aug 3, 2019

Thanks I think I was just being stupid. After this I worked through all the other errors, which I think were mostly just updating py2 to py3. When I try to create a Pull Request the new request is greyed out, is this just me or am I missing something. Thanks for the speedy reply.

@tejt99
Copy link
Contributor Author

tejt99 commented Aug 3, 2019

I have created 3 new pull requests with, I think, everything I changed.

@tejt99
Copy link
Contributor Author

tejt99 commented Aug 3, 2019

Also the pip install doesn't work but I am unsure how that would be fixed

@tejt99
Copy link
Contributor Author

tejt99 commented Aug 3, 2019

Please see the error on #7

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants