Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Wokkel is not working with lastest version of Twisted (21.02) #31

Open
goffi-contrib opened this issue Mar 18, 2021 · 1 comment · May be fixed by #32
Open

Wokkel is not working with lastest version of Twisted (21.02) #31

goffi-contrib opened this issue Mar 18, 2021 · 1 comment · May be fixed by #32
Assignees
@ralphm

Comments

@goffi-contrib
Copy link

Hello,

Wokkel is not working with latest version of Twisted due to the use of twisted.python.compat._PY3. Indeed Twisted has dropped Python 2.7 support in this version, and thus removed _PY3.

Thanks
@micheloosterhof
Copy link 

Python 3.9.0 (default, Oct  8 2020, 06:18:59) 
[Clang 7.1.0 (tags/RELEASE_710/final)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from wokkel import muc
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/michel/src/cowrie/.direnv/python-3.9.0/lib/python3.9/site-packages/wokkel/muc.py", line 25, in <module>
    from wokkel import data_form, generic, iwokkel, xmppim
  File "/Users/michel/src/cowrie/.direnv/python-3.9.0/lib/python3.9/site-packages/wokkel/data_form.py", line 20, in <module>
    from twisted.python.compat import iteritems, unicode, _PY3
ImportError: cannot import name '_PY3' from 'twisted.python.compat' (unknown location)
>>>

@ralphm ralphm linked a pull request May 24, 2021 that will close this issue
Open
@ralphm ralphm self-assigned this May 24, 2021
@ralphm ralphm pinned this issue May 24, 2021
@ralphm ralphm unpinned this issue May 24, 2021
@micheloosterhof micheloosterhof mentioned this issue Oct 14, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ralphm ralphm

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove py2 compat ralphm/wokkel
3 participants
@ralphm @micheloosterhof @goffi-contrib