Re: AC 2.59 incompatibility in getaddrinfo.h

2005-09-08 Thread Paul Eggert
Derek Price [EMAIL PROTECTED] writes:

 shortening the above test to: # if !HAVE_GETADDRINFO, and I'd
 rather just simplify the header

Yes, that sounds right.

I think the defined HAVE_GETADDRINFO cruft dates way back to when we
weren't so sure that the Autoconf macro was part of a distribution
when the .h and .c files were.  Those worries are typically obsolete
nowadays with gnulib.


___
bug-gnulib mailing list
bug-gnulib@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-gnulib


Re: AC 2.59 incompatibility in getaddrinfo.h

2005-09-08 Thread Derek Price
Paul Eggert wrote:

shortening the above test to: # if !HAVE_GETADDRINFO, and I'd
rather just simplify the header



Yes, that sounds right.
  


Okay, thanks, Paul.  I've committed this to CVS CVS for more testing
before I finish importing it into GNULIB.

Cheers,

Derek

-- 
Derek R. Price
CVS Solutions Architect
Ximbiot http://ximbiot.com
v: +1 717.579.6168
f: +1 717.234.3125
mailto:[EMAIL PROTECTED]




___
bug-gnulib mailing list
bug-gnulib@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-gnulib


Re: AC 2.59 incompatibility in getaddrinfo.h

2005-09-08 Thread Simon Josefsson
Derek Price [EMAIL PROTECTED] writes:

 Is there any reason I can't just assume gl_GETADDRINFO ran and config.h
 was included before getaddrinfo.h?  The following test is always coming
 up false on platforms without getaddrinfo (as of AC 2.59, at least,
 AC_CHECK_FUNCS via AC_REPLACE_FUNCS leaves HAVE_GETADDRINFO undefined
 when it is not found):

 # if defined HAVE_GETADDRINFO  !HAVE_GETADDRINFO

 [...decl struct addrinfo  many macros...]

 # endif


 The possible fixes are fixing gl_GETADDRINFO to #define
 HAVE_GETADDRINFO 0 when getaddrinfo isn't found or shortening the above
 test to: # if !HAVE_GETADDRINFO, and I'd rather just simplify the
 header unless there is good reason to leave it as is, especially as
 other naive code with GNULIB installed may be already using #ifdef
 HAVE_GETADDRINFO somewhere.

I used that style in the header file because the automake manual
suggested it, but in gnulib I now think simply using #if
!HAVE_GETADDRINFO would be fine.


___
bug-gnulib mailing list
bug-gnulib@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-gnulib