csringhofer opened a new pull request, #3210: URL: https://github.com/apache/thrift/pull/3210
The error THRIFT-5488 tried to solve resurfaced in a spefic environment: Python3.11.13, chainguard based docker image, GCC 15 Defining PY_SSIZE_T_CLEAN in module.cpp fixed the issue in that environment. Also defined it in types.cpp to consistently define it before including Python.h, which seems to be the recommended way: https://docs.python.org/3.11/c-api/intro.html#include-files It is not clear to me why this was a problem in some environments and not in others. My guess is that including Python.h with different defines leads to different behavior based on GCC version and options. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
