[Patches] [Patch #101437] Fix for 110675: GNU pth support

[noreply@sourceforge.net]

Patch #101437 has been updated. 

Project: 
Category: core (C code)
Status: Open
Summary: Fix for 110675: GNU pth support

Follow-Ups:

Date: 2000-Sep-06 17:01
By: adustman

Comment:
Makes better use of GNU autoconf.

configure --with-pth[=DIR]  # --with-threads implied

Passes test_fork1 and test_thread. Tested with pth-1.3.7.

XXX With pth, select() (and others) do not wake up in response to signals. Example:

>>> import time
>>> time.sleep(10)
[hit ctrl-C, no response for 10 seconds]
Traceback (most recent call last):
  File "<stdin>", line 1, in ?
KeyboardInterrupt
>>>

I have a patch to pth that fixes this that I will try to get incorporated.
-------------------------------------------------------

Date: 2000-Sep-15 10:12
By: gvanrossum

Comment:
Andy,

There's one form issue: you shouldn't patch config.h.in, you should patch acconfig.h. And you should make sure that HAVE_PTH is properly in the alphabet between HAVE_LONG_LONG and HAVE_UINTPTR_T.

Otherwise I don't have an objection. If I don't hear from you soon I may check it in myself. But since I can't test it I prefer to hear it from you!
-------------------------------------------------------

Date: 2000-Sep-15 13:25
By: adustman

Comment:
This patch should fix the style problems. However, I have noticed a new problem: test_poll dumps core:

**Pth** SCHEDULER INTERNAL ERROR: no more thread(s) available to schedule!?!?

I suspect this is purely a pth error.
-------------------------------------------------------

Date: 2000-Sep-15 13:51
By: adustman

Comment:
Note: Support for poll() (in selectmodule) did not exist at the time I came up with the original pth support. However, I neglected to test test_poll when submitting the original version of this patch. Error occurs with both pth-1.3.7 and 1.4a2.
-------------------------------------------------------

Date: 2000-Sep-15 17:30
By: gvanrossum

Comment:
I get autoconf errors with this. I think the AC_CHeCK_PTH macro is undefined.
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=101437&group_id=5470