Avoid including "Python.h" in CPyCppyy/API.h - Githubissues

wlav / CPyCppyy

Other

23 stars 20 forks source link

Avoid including "Python.h" in CPyCppyy/API.h #30

Closed guitargeek closed 4 months ago

guitargeek commented 4 months ago

To really replace TPython for good with the CPyCppyy API, the API needs to forward declare PyObject and avoid using other Python typedefs.

Otherwise, the CPyCppyy/API.h header can't be included in C++ without adding python-specific include paths like /usr/include/python3.12/.

See also:

https://mail.python.org/pipermail/python-dev/2003-August/037601.html

I have opened a PR with the same changes in ROOT to see how the ROOT CI is doing with these changes on all platforms: https://github.com/root-project/root/pull/16019

guitargeek commented 4 months ago

Closing the PR because we don't need these changes in ROOT after all, at least not for the moment