Horton

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

Peter Eisentraut <peter@eisentraut.org>

View thread View on postgr.es
From: Peter Eisentraut <peter@eisentraut.org>
To: Thomas Munro <thomas.munro@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>
Cc: Tomas Vondra <tomas@vondra.me>, Heikki Linnakangas <hlinnaka@iki.fi>, Japin Li <japinli@hotmail.com>, Andres Freund <andres@anarazel.de>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-09-16T09:03:23Z
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

Commits

  1. Move pg_int64 back to postgres_ext.h

  2. pgbench: Make set_random_seed() 64-bit everywhere.

  3. Use PRI?64 instead of "ll?" in format strings (continued).

  4. Fix order of -I switches for building pg_regress.o.

  5. libpq: Deprecate pg_int64.

  6. Use PRI*64 instead of "ll*" in format strings (minimal trial)

  7. Fix header inclusion order in c.h.

  8. Use <stdint.h> and <inttypes.h> for c.h integers.

  9. Remove traces of BeOS.

  10. More correct way to check for existence of types, which allows to specify