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
bpo-40939: Rename PyPegen* functions to PyParser* #21016
Conversation
Renaame PyPegen* functions to PyParser*, so that we can remove the old set of PyParser* functions that were using the old parser.
Seems that there some errors in Windows in Azure |
The windows build still fails, not sure where the errors are coming from :( |
Yeah, me neither. I'm gonna continue investigating, but if someone from the @python/windows-team could enlighten us, it'd be helpful. |
I guess the failure is unrelated to the changes of this PR. It probably has something to do with the Azure environment, since the exact same build target succeeds in the Windows Github Action, but fails with multiple compile errors on Azure. |
But then why other PRs do not have the same problem? |
Unfortunately, I don't know. Will investigate more tomorrow. |
The reason for the breakage is that you're indirectly including |
Or split the header file into a private version and a public one, which I think is cleaner |
And I was wondering why in |
Rename PyPegen* functions to PyParser*, so that we can remove the old set of PyParser* functions that were using the old parser.
Rename PyPegen* functions to PyParser*, so that we can remove the old set of PyParser* functions that were using the old parser.
Renaame PyPegen* functions to PyParser*, so that we can remove the
old set of PyParser* functions that were using the old parser, but keep
everything backwards-compatible.
https://bugs.python.org/issue40939