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.

classification
Title: There is still last bit of GNU Pth code in signalmodule.c
Type: Stage:
Components: Extension Modules Versions: Python 3.11
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: christian.heimes, iritkatriel, jcea, python-dev, vajrasky
Priority: normal Keywords: patch

Created on 2013-08-18 04:58 by vajrasky, last changed 2022-04-11 14:57 by admin.

Files
File name Uploaded Description Edit
remove_gnu_pth.patch vajrasky, 2013-08-18 04:58 review
Messages (4)
msg195542 - (view) Author: Vajrasky Kok (vajrasky) * Date: 2013-08-18 04:58
I read this commit:
http://hg.python.org/cpython/rev/1d5f644b9241

But I noticed there is still GNU Pth code lingering around in Modules/signalmodule.c.

Beside of that the WITH_PTH code (in the same file) is expired already. If you configure python with this option (--with-pth), the configure process will complain that it does not recognize the option.

Attached the patch to clean up this last bit of GNU Pth code.
msg195552 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-08-18 10:43
New changeset 646c2388d8f5 by Christian Heimes in branch 'default':
Issue #18774: Remove last bits of GNU PTH thread code, patch by Vajrasky Kok.
http://hg.python.org/cpython/rev/646c2388d8f5
msg195554 - (view) Author: Christian Heimes (christian.heimes) * (Python committer) Date: 2013-08-18 12:02
Thanks! Three comments still refer to GNU pth:

configure.ac:    # (e.g. gnu pth with pthread emulation)
Python/condvar.h:   library (e.g. gnu pth in pthread emulation) */
Python/thread.c:   library (e.g. gnu pth in pthread emulation) */

thread.c and condvar.h implement a fallback for libraries that don't provide _POSIX_THREADS.
msg407791 - (view) Author: Irit Katriel (iritkatriel) * (Python committer) Date: 2021-12-06 11:43
Two of the three remaining comments Christian mentioned are still there:

% grep -r "gnu pth" Python/ configure.ac
Python//thread.c:   library (e.g. gnu pth in pthread emulation) */
configure.ac:    # (e.g. gnu pth with pthread emulation)
History
Date User Action Args
2022-04-11 14:57:49adminsetgithub: 62974
2021-12-06 11:43:58iritkatrielsetnosy: + iritkatriel

messages: + msg407791
versions: + Python 3.11, - Python 3.4
2013-08-23 19:01:42ezio.melottisetmessages: - msg195643
2013-08-19 16:12:28python-devsetmessages: + msg195643
2013-08-19 05:47:47jceasetnosy: + jcea
2013-08-18 12:02:06christian.heimessetmessages: + msg195554
2013-08-18 10:43:32python-devsetnosy: + python-dev
messages: + msg195552
2013-08-18 04:58:00vajraskycreate