-
Notifications
You must be signed in to change notification settings - Fork 252
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
Fix marker parsing with pyparsing 3.0.5 #487
Conversation
Closes pypa#486 Context in pyparsing: pyparsing/pyparsing#110 (comment)
@9999years @layday we now have competing PRs for the same fix. What are the pro/cons of each? |
The end result is the same, this one's using the expanded callback signature, which isn't necessary since we're only manipulating the token, and item instead of attribute access, which is less transparent. |
Which is to say, feel free to merge either one ;) |
Just a note: having setuptools-scm installed will still break the build with this patch. |
Please use pyparsing 3.0.6, as it restores compatibility with the previous parse action implementation. |
This is with pyparsing 3.0.6 and this patch against packaging version 21.2 With setuptools-scm installed, it looks like this
When I remove setuptools-scm temporarily it will build just fine, with or without the patch |
Closes #486
Context in pyparsing: pyparsing/pyparsing#110 (comment)
Equivalent to #485