-
Notifications
You must be signed in to change notification settings - Fork 41
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
txsocksx dependency prevents use with Python 3 #37
Comments
Good point. There are a huge number number of py2-only things in Foolscap that will need to be fixed before it's py3-capable, so it's kind of a drop in the bucket at this stage, but let's start looking for a py3-safe replacement for the SOCKS functionality. |
I think we're just going to drop support for SOCKS altogether (but retaining support for Tor as you said). It looks like I2P is in the same bucket. @str4d , I don't expect you to be updatding txi2p these days, but do you happen to know of any py3-compatible replacements? |
I think these plans haven't been well-communicated. The txsocksx port is underway - https://github.com/pythonhacker/txsocksx (sponsored by the Tahoe-LAFS foundation). Will the Python 3-capable release of Foolscap certainly drop SOCKS support or is this contingent on the state of the txsocksx port? |
txtorcon dropped txsocksx by growing its own Python2+Python3 SOCKS5 client -- splitting that out would certainly be a possibility (note that it's currently doing custom Tor SOCKS5 opcodes too) |
I am doing the txsocksx port and the tests pass in Python 3.5.6 and Python 2.7.15 now. Welcome to check it out. |
@pythonhacker It seems like it's unclear whether txsocksx is actually a dependency for the Tahoe-LAFS Python 3 port at this point. It's possible it won't be and efforts towards porting txsocksx (and its dependencies) won't actually help the project. Given that, and considering there's plenty of other Tahoe-LAFS porting work left to do, it doesn't seem like this is a good place to spend resources now. |
Anyway, it hardly took much time for me. I restarted this work yesterday and wrapped up the port (at least the tests) today. I was thinking of moving to Foolscap port but it seems like my time is better spent on other libraries. Perhaps we need to take this discussion out of this place. |
I've just landed an initial python3 port on Foolscap's trunk, and to achieve it I removed the socks support (and the dependency on |
It doesn't look like txsocksx is going to be updated for Python 3 (becuase Parsley isn't being updated for Python 3), so having a dependency on that for SOCKS support will preclude the use of Python 3.
txtorcon has already dropped this dependency and so this does not affect Tor support for foolscap.
The text was updated successfully, but these errors were encountered: