Message385355
_zoneinfo starts with:
static int
zoneinfomodule_exec(PyObject *m)
{
PyDateTime_IMPORT;
with:
#define PyDateTime_CAPSULE_NAME "datetime.datetime_CAPI"
#define PyDateTime_IMPORT \
PyDateTimeAPI = (PyDateTime_CAPI *)PyCapsule_Import(PyDateTime_CAPSULE_NAME, 0)
If PyCapsule_Import() fails, zoneinfomodule_exec() returns 0 (success) with an exception raised.
It should check if the import succeeded or not. Concrete example on AIX where datetime cannot be imported:
https://bugs.python.org/issue42604#msg385347
"./python setup.py build" fails with:
Assertion failed: (item != NULL) ^ (PyErr_Occurred() != NULL), file Objects/abstract.c, line 163
--
By the way, the import machinery should raise a SystemError if a module exec function raises an exception *and* reports a success: see _Py_CheckFunctionResult(). |
|
Date |
User |
Action |
Args |
2021-01-20 17:26:06 | vstinner | set | recipients:
+ vstinner |
2021-01-20 17:26:06 | vstinner | set | messageid: <1611163566.26.0.253539458839.issue42979@roundup.psfhosted.org> |
2021-01-20 17:26:06 | vstinner | link | issue42979 messages |
2021-01-20 17:26:06 | vstinner | create | |
|