[WIP] bpo-35081: _testcapi is now compiled with Py_BUILD_CORE#10274
[WIP] bpo-35081: _testcapi is now compiled with Py_BUILD_CORE#10274vstinner wants to merge 1 commit intopython:masterfrom vstinner:testcapi_build_core
Conversation
* setup.py now defines Py_BUILD_CORE to build the _testcapi module (_testcapimodule.c). * pycore_pathconfig.h now requires Py_BUILD_CORE to be defined.
|
IHMO this PR depends on PR #10238 to properly define datetime.h when compiling with Py_BUILD_CORE. |
|
Is not intended to build it without Py_BUILD_CORE for testing the same C API that is available in external extensions? |
Honestly, the exact purpose of _testcapi is now unclear to me :-) This module became giant. Maybe we need a new module to the test the Py_BUILD_CORE API? _testcapi.get_coreconfig() converts PyInterpreterState.core_config to a Python function, from my point of view, this code is not part of the public Python C API. -- Other options:
|
|
I'm going to split If the only problem is with |
Right. I abandon this PR and wrote PR #10362 which is the proper fix: it also ensures that _testcapimodule.c is not compiled with Py_BUILD_CORE defined. |
(_testcapimodule.c).
https://bugs.python.org/issue35081