Re: apr 1.4: testpoll crash on OSX 10.6

2009-10-24 Thread Neil Conway
Attached is a patch against trunk that fixes this problem by changing the test suite. For some of the tests, it is sufficient to change the poll() timeout from 0 (one-time poll) to -1 (blocking poll). In a few other places, the semantics of the test needed to be changed -- e.g. if we do a blocking

Re: apr 1.4: testpoll crash on OSX 10.6

2009-10-24 Thread Neil Conway
On a related note, ISTM that many of the tests for the poll / pollset features are wrong in principle. They apparently assume that if you send a UDP datagram to localhost and then immediately poll() for it (with a timeout of zero), the poll() will pickup the UDP datagram you just sent. That is not

Crash in apr_pollcb_create_ex()

2009-10-24 Thread Neil Conway
If apr_pollcb_create_ex() is called with a non-default pollset method, it tries to use the user-specified method, and fails back to the platform default if the user-specified method is not available on the current platform. The fall-back code path is buggy, however: the *pollcb pointer is reset to

Re: svn commit: r821306 - /apr/apr/trunk/include/apr_errno.h

2009-10-24 Thread Branko Čibej
Ping? I really would appreciate any kind of recollection as to why we haven't added this mapping ages ago -- apart from the obvious reason of my dropping the ball then. -- Brane Branko Čibej wrote: > Back in 2002 I raised a question [1] about mapping the Windows error > code ERROR_DIRECTORY into

Re: Mac OS X Universal builds and APR

2009-10-24 Thread Jeff Trawick
On Sat, Oct 24, 2009 at 8:16 AM, Barry Scott wrote: > > On 22 Oct 2009, at 20:47, Barry Scott wrote: > >> >> On 20 Oct 2009, at 22:30, Jeff Trawick wrote: >> >>> On Tue, Oct 20, 2009 at 4:54 PM, Barry Scott >>> wrote: On 20 Oct 2009, at 04:18, Branko Čibej wrote: > Barry Scott

Re: Mac OS X Universal builds and APR

2009-10-24 Thread Barry Scott
On 22 Oct 2009, at 20:47, Barry Scott wrote: On 20 Oct 2009, at 22:30, Jeff Trawick wrote: On Tue, Oct 20, 2009 at 4:54 PM, Barry Scott emacs.org> wrote: On 20 Oct 2009, at 04:18, Branko Čibej wrote: Barry Scott wrote: I think the problem is that configure is used to find out things t