Re: [GENERAL] [PATCH] Better way to check for getaddrinfo

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: [GENERAL] [PATCH] Better way to check for getaddrinfo
Дата
Msg-id 200601261849.k0QInY012282@candle.pha.pa.us
обсуждение исходный текст
Ответ на Re: [GENERAL] [PATCH] Better way to check for getaddrinfo  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [GENERAL] [PATCH] Better way to check for getaddrinfo  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Tom Lane wrote:
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > I am not sure what to do on this.  Right now we have a one-line test:
> >     AC_REPLACE_FUNCS([getaddrinfo])
> > To test for a macro we are going to need to add include netdb.h, and the
> > LINK test below is overkill.  I am thinking we should just hard-code in
> > HAVE_GETADDRINFO for the True64 platform;  anything more is going to be
> > just a Tru64 hack anyway.
>
> I still want to understand why any change is needed at all.  There must
> be something very peculiar about getaddrinfo on Tru64 if the original
> coding doesn't work.  Why is it different from every other function we
> test for?

I have the answer.  Tru64 netdb.h has:

    #if defined (_SOCKADDR_LEN) || defined (_XOPEN_SOURCE_EXTENDED)
    #define getaddrinfo ngetaddrinfo
    #else
    #define getaddrinfo ogetaddrinfo
    #endif

so it is a macro, and configure produces this line:

    #undef $ac_func

meaning that even if we added #include <netdb.h>, our configure test
still would not work.

Perhaps we should just test for ngetaddrinfo on that platform, and
define HAVE_GETADDRINFO.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: [GENERAL] [PATCH] Better way to check for getaddrinfo
Следующее
От: Tom Lane
Дата:
Сообщение: Re: [GENERAL] [PATCH] Better way to check for getaddrinfo