Adjust imports for Python 3.11 #1140
Conversation
sre_parse and sre_constants are deprecated in 3.11, see python/cpython#91308
lark/utils.py
Outdated
| @@ -113,8 +113,13 @@ def deserialize(cls, data, namespace, memo): | |||
| except ImportError: | |||
| regex = None | |||
|
|
|||
| import sre_parse | |||
| import sre_constants | |||
| try: # Python 3.11 | |||
There was a problem hiding this comment.
I recommend using sys.version_info >= (3, 11) or similar instead of try on import, it serves as documentation to the user, it enables the workaround to be dropped when that version of Python is dropped, and it can be used by static analysis tools like mypy and pyupgrade.
There was a problem hiding this comment.
Adjusted accordingly. Also has its downsides, though (it will break now with Python 3.11 before alpha 7, and there isn't a very straightforward way to check for that).
There was a problem hiding this comment.
True, though older alpha / beta / rcs are not worth supporting after a new one comes out. Especially alphas once a beta is out. They are essentially just development snapshots.
Build 0.8.0 doesn’t support alpha 6 either.
|
Thanks for contributing! |
|
Any timeline on a release? As far as I can tell this is the last thing stopping me from testing with |
sre_parse and sre_constants are deprecated in 3.11, see python/cpython#91308
This fixes a
DeprecationWarningwhen runningpython3 -Wall -c "import lark"with Python 3.11.0a7.I couldn't run the tests on Python 3.11 due to PiotrDabkowski/Js2Py#282, but it looks like at least lark works without deprecation warnings with this change.