You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Also, we should define $env:IncludeUWP="true" in the Windows PR tests. That will build extra binaries every time (that we ship in the Windows Store) and would catch this issue earlier.
Activity
gvanrossum commentedon Nov 24, 2021
E.g. https://dev.azure.com/Python/cpython/_build/results?buildId=92084&view=results
zooba commentedon Nov 24, 2021
Seems we lost the C++ handling in Include/cpython/initconfig.h:
<full body of file>
zooba commentedon Nov 24, 2021
Also, we should define $env:IncludeUWP="true" in the Windows PR tests. That will build extra binaries every time (that we ship in the Windows Store) and would catch this issue earlier.
gvanrossum commentedon Nov 24, 2021
Christian thinks that a PR by Victor broke this (GH-29488 / bpo-39026)
tiran commentedon Nov 24, 2021
New changeset f4afc53 by Christian Heimes in branch 'main':
bpo-45893: Add missing extern C to initconfig.h (GH-29761)
f4afc53
vstinner commentedon Nov 24, 2021
I was never in initconfig.h since the PEP-587 (PyConfig C API) was implemented. I'm not sure why the build *started* to fail on Windows.
Thanks for fixing the issue ;-)
miss-islington commentedon Nov 24, 2021
New changeset cd85d91 by Miss Islington (bot) in branch '3.10':
bpo-45893: Add missing extern C to initconfig.h (GH-29761)
cd85d91
tiran commentedon Nov 24, 2021
You are welcome!