Re: [HACKERS] Proposed patch to getaddrinfo.c to support

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Petr Jelinek <pjmodos(at)seznam(dot)cz>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-patches(at)postgresql(dot)org, dpage(at)vale-housing(dot)co(dot)uk, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Chuck McDevitt <cmcdevitt(at)greenplum(dot)com>
Subject: Re: [HACKERS] Proposed patch to getaddrinfo.c to support
Date: 2005-08-26 11:54:17
Message-ID: 430F02E9.50807@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Petr Jelinek wrote:

> Andrew Dunstan wrote:
>
>> I suspected we'd forgotten something.
>>
>> The attached small patch appears to be what's required (at least on
>> loris). "make check" failed but not for any apparent ipv6 reason.
>> More importantly, we correctly set HAVE_IPV6 and HAVE_STRUCT_ADDRINFO.
>
>
> Well this is what I ment with those proposed changes - I haven't sent
> patch, just said whats needed - always define HAVE_IPV6 and
> HAVE_STRUCT_ADDRINFO and include ws2tcpip.h, if you do just that
> include like your patch did, you'll break building on W2k (and like I
> said I don't know how to make HAVE_IPV6 and HAVE_STRUCT_ADDRINFO
> always defined under windows because I am not familiar with configure
> and thats why I haven't sent patch).
>

Really? Please don't assert it, test it and tell us what the error is. I
find it highly unlikely that it will break building on w2k, and only
slightly less unlikely that it will break running on w2k. But I want to
see the evidence (make error or error from runtime log).

The patch I sent should be exactly what is required to have HAVE_IPV6
and HAVE_STRUCT_ADDRINFO defined on windows. That should be true
regardless of which windows you are building on - the headers should be
the same.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Glaesemann 2005-08-26 12:01:13 Re: [ANNOUNCE] Welcome Core Team member Dave Page
Previous Message Tino Wildenhain 2005-08-26 11:41:08 Re: [ANNOUNCE] Welcome Core Team member Dave Page

Browse pgsql-patches by date

  From Date Subject
Next Message Petr Jelinek 2005-08-26 12:28:42 Re: [HACKERS] Proposed patch to getaddrinfo.c to support
Previous Message Petr Jelinek 2005-08-26 10:22:31 Re: [HACKERS] Proposed patch to getaddrinfo.c to support