On Tue, 25 Mar 2008 22:41:56 -0500, "Craig A. Berry" <[EMAIL PROTECTED]>
wrote:

> At 9:27 AM +0100 3/24/08, H.Merijn Brand wrote:
> >On Sun, 23 Mar 2008 14:29:25 -0500, "John E. Malmberg" <[EMAIL PROTECTED]> 
> >wrote:
> >
> > > In configure.com, add test for sys/poll.h.
> 
> 
> > Thanks John, applied as change #33550
> 
> Thanks for keeping up with VMS patches while my attention was elsewhere.
> 
> > FYI, this test was added by the updated metaconfig tools, not by any
> > patch from the perl community, so it missing in any of the OS's that
> > do not use Configure was not a big deal.
> >
> > However, this is good housekeeping. I'd love to see the remaining OS
> > variants updated too.
> 
> On Mac OS X, I'm seeing:
> 
> <poll.h> found.
> Hmm.  Based on the hints in hints/darwin.sh,
> the recommended value for $i_poll on this machine was "undef"!
>     Keep the recommended value? [y]
> 
> Do all the hints files need to be updated?  The IO extension includes
> poll.h unconditionally, so pretty much everything must have it.

I obviously underestimated the effect  of this automatically being added
by the new configuration stuff.

If poll.h is automatically always included, we are either doing it wrong
or we have no alternatives, in which case the check would be void. I can
remove it if that would be preferred.

-- 
H.Merijn Brand         Amsterdam Perl Mongers (http://amsterdam.pm.org/)
using & porting perl 5.6.2, 5.8.x, 5.10.x  on HP-UX 10.20, 11.00, 11.11,
& 11.23, SuSE 10.1 & 10.2, AIX 5.2, and Cygwin.       http://qa.perl.org
http://mirrors.develooper.com/hpux/            http://www.test-smoke.org
                        http://www.goldmark.org/jeff/stupid-disclaimers/

Reply via email to