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.

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

Issues running test suite under Python 3.9 #1772

Closed
sethmlarson opened this issue Dec 28, 2019 · 3 comments · Fixed by #1775
Closed

Issues running test suite under Python 3.9 #1772

sethmlarson opened this issue Dec 28, 2019 · 3 comments · Fixed by #1775

Comments

@sethmlarson
Copy link
Member

Tornado is an issue due to the collections.abc change switching from deprecation warnings to errors in 3.9. Unfortunately moving to the latest Tornado version also doesn't work and might be more work than it's worth. I'm thinking if the issue is only to do with collections.abc we vendor the library and make the modification ourselves.

@tirkarthi
Copy link

Is there a log of the warnings emitted? Quick search of tornado code doesn't show any instances of using ABC from collections module directly.

@sethmlarson
Copy link
Member Author

@tirkarthi It's the specific version we're using, we're pinned <6 for now because migrating breaks a lot of tests. We could go the other way towards trying to upgrade to the latest Tornado but I feel like that's a lot more work than vendoring 5.1.1.

@pquentin
Copy link
Member

pquentin commented Jan 6, 2020

we're pinned <6 for now because migrating breaks a lot of tests.

The issue is that Tornado 6+ does not support Python 2.7 because it relies on asyncio. But I just checked and Tornado 6+ works fine on Python 3.5+ (including 3.9). Since Tornado 6 and 5 share the same API for our purposes, I think the better option is to use Tornado 6 on Python 3 and Tornado 5 on Python 2. That would be a two lines change. :)

If you're OK with this I'll open a PR once the build is fixed by #1774.

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

Successfully merging a pull request may close this issue.

3 participants