➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2021-11-24 18:50 by gvanrossum, last changed 2022-04-11 14:59 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 29761 | merged | christian.heimes, 2021-11-24 19:20 | |
| PR 29765 | merged | miss-islington, 2021-11-24 20:12 | |
| Messages (8) | |||
|---|---|---|---|
| msg406945 - (view) | Author: Guido van Rossum (gvanrossum) *  |
Date: 2021-11-24 18:50 | |
E.g. https://dev.azure.com/Python/cpython/_build/results?buildId=92084&view=results |
|||
| msg406946 - (view) | Author: Steve Dower (steve.dower) * |
Date: 2021-11-24 18:51 | |
Seems we lost the C++ handling in Include/cpython/initconfig.h: #ifdef __cplusplus extern "C" { #endif <full body of file> #ifdef __cplusplus } #endif |
|||
| msg406947 - (view) | Author: Steve Dower (steve.dower) * |
Date: 2021-11-24 18:59 | |
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. |
|||
| msg406950 - (view) | Author: Guido van Rossum (gvanrossum) * |
Date: 2021-11-24 19:03 | |
Christian thinks that a PR by Victor broke this (GH-29488 / bpo-39026) |
|||
| msg406956 - (view) | Author: Christian Heimes (christian.heimes) * |
Date: 2021-11-24 20:12 | |
New changeset f4afc53bf68c8ded20b281cd1baa88a679b4a3fd by Christian Heimes in branch 'main': bpo-45893: Add missing extern C to initconfig.h (GH-29761) https://github.com/python/cpython/commit/f4afc53bf68c8ded20b281cd1baa88a679b4a3fd |
|||
| msg406963 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2021-11-24 21:33 | |
> Seems we lost the C++ handling in Include/cpython/initconfig.h 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 ;-) |
|||
| msg406965 - (view) | Author: miss-islington (miss-islington) | Date: 2021-11-24 21:57 | |
New changeset cd85d91bc66a587ce2ba668c897a5ecf118733cc by Miss Islington (bot) in branch '3.10': bpo-45893: Add missing extern C to initconfig.h (GH-29761) https://github.com/python/cpython/commit/cd85d91bc66a587ce2ba668c897a5ecf118733cc |
|||
| msg406966 - (view) | Author: Christian Heimes (christian.heimes) * |
Date: 2021-11-24 22:00 | |
You are welcome! |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:52 | admin | set | github: 90051 |
| 2021-11-24 22:00:12 | christian.heimes | set | status: open -> closed resolution: fixed messages: + msg406966 stage: patch review -> resolved |
| 2021-11-24 21:57:52 | miss-islington | set | messages: + msg406965 |
| 2021-11-24 21:33:14 | vstinner | set | messages: + msg406963 |
| 2021-11-24 20:12:20 | miss-islington | set | keywords:
+ patch nosy: + miss-islington pull_requests: + pull_request28003 |
| 2021-11-24 20:12:16 | christian.heimes | set | messages: + msg406956 |
| 2021-11-24 19:21:13 | christian.heimes | set | keywords:
- patch type: compile error versions: + Python 3.10 |
| 2021-11-24 19:20:15 | christian.heimes | set | keywords:
+ patch stage: patch review pull_requests: + pull_request27999 |
| 2021-11-24 19:03:36 | gvanrossum | set | nosy:
+ vstinner messages: + msg406950 |
| 2021-11-24 18:59:55 | steve.dower | set | messages: + msg406947 |
| 2021-11-24 18:51:34 | steve.dower | set | keywords:
+ easy (C) messages: + msg406946 versions: + Python 3.11 |
| 2021-11-24 18:50:48 | gvanrossum | set | nosy:
+ christian.heimes |
| 2021-11-24 18:50:21 | gvanrossum | create | |