python-cython C++ support patch

Marco Atzeri marco.atzeri@gmail.com
Sat Jan 30 13:58:42 GMT 2021


On 30.01.2021 14:28, Masamichi Hosoda wrote:
>> have you considered that you just need to define
>> CYTHON_NO_PYINIT_EXPORT ?
>>
>> the portion of the code below your change has already the
>> ifdef __cplusplus semantic
>>
>> Have you proposed it upstream ? It does not seem
>> a change restricted to Cygwin
>>
>> Any way I see no "wrongness" to add it on the Cython rebuild
> 
> Hi Marco,
> 
> The relevant upstream source looks like this.
> https://github.com/cython/cython/blob/0.29.21/Cython/Utility/ModuleSetupCode.c#L712
> 
> ```
> #define __Pyx_PyMODINIT_FUNC PyMODINIT_FUNC

PyMODINIT_FUNC unfortunately dos not work and my changes try
to overcome it, see similar on

https://sourceware.org/pipermail/cygwin/2021-January/247211.html

but your portion is for additional C++ case
than I am adding in asimilar way

Regards
Marco





More information about the Cygwin mailing list