➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Roman Yurchak
Recipients Roman Yurchak, pablogsal, serhiy.storchaka
Date 2020-08-06.20:20:07
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1596745207.18.0.893678923913.issue41498@roundup.psfhosted.org>
In-reply-to
Content 
The `_Py_Sigset_Converter` function is conditionally defined  in https://github.com/python/cpython/blob/777b611c8c5676b80898a429f71d28e59bddc49d/Modules/posixmodule.h#L27 if `ifdef HAVE_SIGSET_T`

However this function is called unconditionally in https://github.com/python/cpython/blob/777b611c8c5676b80898a429f71d28e59bddc49d/Modules/clinic/signalmodule.c.h#L385 (there are 4 such occurrences), which leads to a compilation error when `HAVE_SIGSET_T` is not set.

This is regression in 3.8+ as far as I can tell due to changes in https://github.com/python/cpython/pull/6720 

I imagine the solution could be to always define this function but raise an exception if it is called when HAVE_SIGSET_T` is undefined, as done in the following patch: https://github.com/iodide-project/pyodide/blob/fc5495ffdb54a11fd588dc60ba6b8777f90c3724/cpython/patches/0006-fix-Py_Sigset_Converter.patch 

Would this be acceptable, or is there a better way of handling it? Thanks!
History
Date User Action Args
2020-08-06 20:20:07Roman Yurchaksetrecipients: + Roman Yurchak, serhiy.storchaka, pablogsal
2020-08-06 20:20:07Roman Yurchaksetmessageid: <1596745207.18.0.893678923913.issue41498@roundup.psfhosted.org>
2020-08-06 20:20:07Roman Yurchaklinkissue41498 messages
2020-08-06 20:20:07Roman Yurchakcreate