On Thu, 2005-10-06 at 12:05 -0500, Albert Chin wrote:
> On Thu, Oct 06, 2005 at 12:55:26PM +0100, Dave Shield wrote:


> > Assuming that IRIX *does* ship with <netinet/{tcp,ip}.h>,
> > then explicitly defining the corresponding tokens can't
> > do any harm (though perhaps these lines should be protected
> > by suitable #ifndef checks?)
> 
> Regardless of whether or not it ships with <netinet/{tcp,ip}.h>, the
> AC_CHECK_HEADERS test will catch it.

It should do, yes.
But the comment in the irix.h file seems to indicate that
(at least on some versions of IRIX), these tests weren't working
properly - hence needing to hardwire the "correct" results.

It may be that later amendments to the configure script may
have fixed this problem - but a bug-fix release isn't the time
to find out.

Thomas has applied your patch to the main development code,
so the 5.3 release process should hopefully catch any problems
with earlier IRIX systems (assuming there's someone who can
try this out!)



> > The inet_addr() and inet_ntoa() checks seem to be responding
> > to specific IRIX problems.  Simply removing these settings without
> > further investigation is probably unwise.
> 
> It's defined in <arpa/inet.h> on 6.5.23m:
>   extern char *inet_ntoa(struct in_addr);
> 
> It doesn't cause a problem during the build. I don't know if it is
> needed for an earlier IRIX release.

Checking the history, the suppression of <arpa/inet.h> dates
from November 2001  (as does the explicit inclusion of
<netinet/{tcp,ip}.h>) - though the explanation was only added
some time later.
  The explicit declaration of 'inet_ntoa()' is clearly a
consequence of not including <arpa/inet.h>


What is the definition of inet_addr() in <arpa/inet.h> ?


Dave


-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to