-
-
Notifications
You must be signed in to change notification settings - Fork 58
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
Python 3.9: DeprecationWarning: The parser module is deprecated, tests fail #310
Comments
The parser module is deprecated on Python 3.9 Fixes sqlalchemy#310 The same change needs to be done in babel
The parser module is deprecated on Python 3.9 Fixes sqlalchemy/mako#310 See also sqlalchemy/mako#311
OK so for fedora packaging you can add a patch to your specfile for now, correct ? can you link me to the redhat BZ ? |
Mike Bayer has proposed a fix for this issue in the master branch: Replace usage of parser.suite with ast.parse https://gerrit.sqlalchemy.org/1654 |
if you want to give the thumbs up on that I can release tomorrow or something |
https://bugzilla.redhat.com/show_bug.cgi?id=1787962 Patching is fine, no need for a quick release. I had an idea that the tests failed on 2.7 without the ImportError, but there might have been a typo or something like that. |
Replaced usage of the long-superseded "parser.suite" module in the mako.util package for parsing the python magic encoding comment with the "ast.parse" function introduced many years ago in Python 2.5, as "parser.suite" is emitting deprecation warnings in Python 3.9. Fixes sqlalchemy/mako#310 See also sqlalchemy/mako@2dae7d2
Replaced usage of the long-superseded "parser.suite" module in the mako.util package for parsing the python magic encoding comment with the "ast.parse" function introduced many years ago in Python 2.5, as "parser.suite" is emitting deprecation warnings in Python 3.9. Fixes sqlalchemy/mako#310 See also sqlalchemy/mako@2dae7d2
OK that is released on pypi in 1.1.1. |
Hello. In Fedora, we are trying to build packages with Python 3.9.0a2. mako 1.1.0 tests fail with:
The text was updated successfully, but these errors were encountered: