From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Tomas Vondra <tomas(at)vondra(dot)me>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Japin Li <japinli(at)hotmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Cannot find a working 64-bit integer type on Illumos |
Date: | 2025-09-16 09:03:23 |
Message-ID: | b0870e2f-2b41-495d-b225-75b7991a1d12@eisentraut.org |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 12.09.25 15:51, Peter Eisentraut wrote:
> On 04.09.25 11:01, Peter Eisentraut wrote:
>> On 03.09.25 17:04, Peter Eisentraut wrote:
>>> Consider a third-party extension that does something like dblink or
>>> postgres_fdw. It will compile against a server and also a libpq.
>>> The server and the libpq might not be of the same major version. (On
>>> Debian, only the latest libpq will be available.) If you have for
>>> example server version 17 and libpq version 18, then you will get the
>>> pg_int64 typedef both from postgres_ext.h (from the PG17 server
>>> includes) and from libpq-fe.h (from PG18 libpq). That is not allowed
>>> in C99, and even if it were, the underlying types of PG_INT64_TYPE
>>> (in PG17) and int64_t (in PG18) might be different (long int vs. long
>>> long int) and this would fail.
>>>
>>> I think this could be fixed by moving the definition of pg_int64 back
>>> to postgres_ext.h. Then extension builds would only get one
>>> definition, because of the header guards. Depending on include
>>> order, they could get a different underlying type, but that's a
>>> smaller problem, since the type is supposed to be deprecated anyway.
>>
>> Here is a patch that has been reported to fix the problem.
>
> I propose to go ahead with this patch in a few days if there are no
> other solutions coming.
done
From | Date | Subject | |
---|---|---|---|
Next Message | Aleksander Alekseev | 2025-09-16 09:08:57 | Re: [PATCH] Refactor bytea_sortsupport(), take two |
Previous Message | jian he | 2025-09-16 08:52:02 | Re: Add SPLIT PARTITION/MERGE PARTITIONS commands |