Re: provide inet_?to? declarations in arpa_inet.h

Bruno Haible [EMAIL PROTECTED] writes: The variable are set to 0 in the inet_ntop/inet_pton modules, via a AC_CHECK_DECLS statement. No, it isn't. AC_CHECK_DECLS([inet_ntop]...) sets the shell variable ac_cv_have_decl_inet_ntop to yes or no. It does not set the shell variable

Re: provide inet_?to? declarations in arpa_inet.h

Simon Josefsson wrote: +-e 's|@''HAVE_DECL_INET_NTOP''@|$(HAVE_DECL_INET_NTOP)|g' \ +-e 's|@''HAVE_DECL_INET_PTON''@|$(HAVE_DECL_INET_PTON)|g' \ $(srcdir)/arpa_inet.in.h; \ } [EMAIL PROTECTED] mv [EMAIL PROTECTED] $@ This cannot be correct/complete.

Re: provide inet_?to? declarations in arpa_inet.h

Bruno Haible [EMAIL PROTECTED] writes: Simon Josefsson wrote: diff --git a/m4/arpa_inet_h.m4 b/m4/arpa_inet_h.m4 index 8f530c5..baddf0d 100644 --- a/m4/arpa_inet_h.m4 +++ b/m4/arpa_inet_h.m4 @@ -32,4 +32,7 @@ AC_DEFUN([gl_ARPA_INET_H_DEFAULTS], [ GNULIB_INET_NTOP=0;

Re: provide inet_?to? declarations in arpa_inet.h

Simon Josefsson [EMAIL PROTECTED] writes: Bruno Haible [EMAIL PROTECTED] writes: Hi Simon, I would also favor removing both inet_ntop.h and inet_pton.h, and use arpa/inet.h for the declarations, as you suggested. Me too. Yoann, Bruno, I've pushed the following, please test it. I've

Re: provide inet_?to? declarations in arpa_inet.h

Simon Josefsson wrote: I've tested it better now It was still apparently tested only on few systems. As you can see from doc/posix-functions/inet*.texi, the functions inet_ntop and inet_pton need to be declared also on HP-UX 11, OSF/1 4.0, Solaris 2.5.1. So the gnulib arpa/inet.h must be used in

Re: provide inet_?to? declarations in arpa_inet.h

Bruno Haible [EMAIL PROTECTED] writes: Simon Josefsson wrote: I've tested it better now It was still apparently tested only on few systems. As you can see from doc/posix-functions/inet*.texi, the functions inet_ntop and inet_pton need to be declared also on HP-UX 11, OSF/1 4.0, Solaris

Re: provide inet_?to? declarations in arpa_inet.h

Simon Josefsson wrote: Btw, a solaris system I have access to declares the function as follows (in system arpa/inet.h): #if !defined(_XPG4_2) || defined(_XPG6) || defined(__EXTENSIONS__) extern int inet_pton(int, const char *_RESTRICT_KYWD, void *_RESTRICT_KYWD); extern const char

Re: provide inet_?to? declarations in arpa_inet.h

Simon Josefsson wrote: diff --git a/m4/arpa_inet_h.m4 b/m4/arpa_inet_h.m4 index 8f530c5..baddf0d 100644 --- a/m4/arpa_inet_h.m4 +++ b/m4/arpa_inet_h.m4 @@ -32,4 +32,7 @@ AC_DEFUN([gl_ARPA_INET_H_DEFAULTS], [ GNULIB_INET_NTOP=0; AC_SUBST([GNULIB_INET_NTOP]) GNULIB_INET_PTON=0;

Re: provide inet_?to? declarations in arpa_inet.h

Hi Simon, I would also favor removing both inet_ntop.h and inet_pton.h, and use arpa/inet.h for the declarations, as you suggested. Me too. Bruno

Re: provide inet_?to? declarations in arpa_inet.h

Yoann Vandoorselaere [EMAIL PROTECTED] writes: Hi Simon, Le mardi 22 avril 2008 à 10:54 +0200, Simon Josefsson a écrit : Gnulib's current (minimal) arpa/inet.h for MinGW doesn't declare inet_ntop and inet_pton, which doesn't exist on MinGW. However, gnulib has inet_ntop and inet_pton

provide inet_?to? declarations in arpa_inet.h

Hi, Gnulib's current (minimal) arpa/inet.h for MinGW doesn't declare inet_ntop and inet_pton, which doesn't exist on MinGW. However, gnulib has inet_ntop and inet_pton modules with their own header files. This patch makes sure that arpa/inet.h provides the expected declarations. Comments?

Re: provide inet_?to? declarations in arpa_inet.h

Hi Simon, Le mardi 22 avril 2008 à 10:54 +0200, Simon Josefsson a écrit : Gnulib's current (minimal) arpa/inet.h for MinGW doesn't declare inet_ntop and inet_pton, which doesn't exist on MinGW. However, gnulib has inet_ntop and inet_pton modules with their own header files. This patch makes