Re: Cannot find a working 64-bit integer type on Illumos

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

In response to

Browse pgsql-hackers by date

  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