Re: msys inet_pton strangeness

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: msys inet_pton strangeness
Date: 2024-09-28 17:26:21
Message-ID: 4033d12c-2d82-470e-ab8c-3ab2f42985bb@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 2024-09-28 Sa 11:49 AM, Tom Lane wrote:
> Andrew Dunstan<andrew(at)dunslane(dot)net> writes:
>> It's complaining like this:
>> C:/tools/xmsys64/home/pgrunner/bf/root/REL_15_STABLE/pgsql/src/interfaces/libpq/fe-secure-common.c:219:21: error: implicit declaration of function 'inet_pton'; did you mean 'inet_aton'? [-Wimplicit-function-declaration]
>> 219 | if (inet_pton(AF_INET6, host, &addr) == 1)
>> | ^~~~~~~~~
>> configure has determined that we have inet_pton, and I have repeated the
>> test manually.
> configure's test is purely a linker test. It does not check to see
> where/whether the function is declared. Meanwhile, the compiler is
> complaining that it doesn't see a declaration. So the problem
> probably can be fixed by adding an #include, but you'll need to
> figure out what.
>
> I see that our other user of inet_pton, fe-secure-openssl.c,
> has a rather different #include setup than fe-secure-common.c;
> does it compile OK?

I'll try, but this error occurs before we get that far.

We should have included ws2tcpip.h, which includes this:

#define InetPtonA inet_pton
WINSOCK_API_LINKAGE INT WSAAPI InetPtonA(INT Family, LPCSTR pStringBuf, PVOID pAddr);

It's conditioned on (_WIN32_WINNT >= 0x0600), but that should be true.

So I'm still very confused ;-(

cheers

andrew

--
Andrew Dunstan
EDB:https://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2024-09-28 19:45:36 Re: Better error reporting from extension scripts (Was: Extend ALTER OPERATOR)
Previous Message Sutou Kouhei 2024-09-28 15:56:45 Re: Make COPY format extendable: Extract COPY TO format implementations