Uh oh!
There was an error while loading. Please reload this page.
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.
pablogsal
commented
Jun 20, 2020
Seems that there some errors in Windows in Azure |
pablogsal
commented
Jun 21, 2020
The windows build still fails, not sure where the errors are coming from :( |
lysnikolaou
commented
Jun 21, 2020
Yeah, me neither. I'm gonna continue investigating, but if someone from the @python/windows-team could enlighten us, it'd be helpful. |
lysnikolaou
commented
Jun 21, 2020
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. |
pablogsal
commented
Jun 21, 2020
But then why other PRs do not have the same problem? |
lysnikolaou
commented
Jun 21, 2020
Unfortunately, I don't know. Will investigate more tomorrow. |
nnemkin
commented
Jun 21, 2020
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 |
lysnikolaou
commented
Jun 21, 2020
And I was wondering why in |
Uh oh!
There was an error while loading. Please reload this page.
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