-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Change documentation for optional dependency SOAPpy to SOAPpy-py3 #12145
Comments
Thanks for the report. I don't think that we use soappy anymore as a dependency https://github.com/twisted/twisted/blob/trunk/pyproject.toml But I do see that soappy is imported. https://github.com/twisted/twisted/blob/trunk/src/twisted/web/soap.py The source code for the documentation is at My suggestion is to just deprecated the soap support in Twisted. And the soap dependency was not defined for more than 4 years https://github.com/twisted/twisted/blob/80b02d87adc48f813179aaa31ff5aaaa574200e6/setup.cfg I could not find where the soap dependency was first defined. Do you use the SOAP code with twisted ? |
Looking again at my code, we do use SOAP with twisted, but we don't use the What you say makes total sense :) To deprecate the soap module in twisted since I'm pretty sure it won't work on Python 3.10+. I started a few months ago to port a project from Python 2.7 to 3.10, and SOAPpy did not handle it well (especially one of its dependencies wstools). If I recall correctly it was mainly due to changes in Python's standard library like |
Thanks for the feedback. I think that is safe to just remove the SOAP code It is already broken. It looks like we still have code that was not ported to py3 ... or for which we don't execute the tests. If anyone want to have SOAP support in Twisted, they can send a PR
|
Normally I pop up to say how great some weird obscure ancient Twisted module is, but honestly the SOAP code was a bit of a sad afterthought even at the time, and clearly it doesn't really work now. I don't know if anyone ever used it. Obviously, do the usual mailing list thing, but unless there's some strenuous objection, let's dump it. |
I plan to use this ticket to update the instalallation documentation. I have created a separate ticket to track the removal of SOAP - #12146 I will send a message to the mailing list one the SOAP removal PR is ready to review |
A minor maintenance issue. SOAPpy seems to be no longer maintained and does not support newest Python 3 releases. I've been using this fork instead SOAPpy-py3 which seems to be actively maintained. While I don't use all the features, the transition was painless and I didn't need to change any of my own code.
For reference, this is the documentation I'm talking about: https://docs.twisted.org/en/latest/installation/howto/optional.html
The text was updated successfully, but these errors were encountered: