re: Comments for a patch?

[Dan Kegel]

I'm not an autoconf guru, but I play one on TV.

You wrote:

+AC_CHECK_FUNC(getaddrinfo, [AC_DEFINE([HAVE_GETADDRINFO], [1],
+    [Defined to 1 if you have the getadrinfo function,])],
+    [AC_CHECK_LIB(socket, getaddrinfo,
+    [AC_DEFINE([HAVE_GETADDRINFO]) LIBS="-lsocket -lnsl $LIBS"], , , -lnsl)])

I'm not sure it's healthy to assume that -lsocket and -lnsl go
together like that.  Have you looked at how Samba does it?

See http://samba.org/ftp/unpacked/distcc/configure.ac

- Dan