public inbox for [email protected]  

help / color / mirror / Atom feed

From: Peter Eisentraut <[email protected]>
To: Srirama Kucherlapati <[email protected]>
To: Andres Freund <[email protected]>
To: AIX PG user <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: AIX support
Date: Fri, 7 Nov 2025 17:28:13 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <SJ4PPFB8177832600E5AAE4214587DC3F7BDBE8A@SJ4PPFB81778326.namprd15.prod.outlook.com>
References: <CY5PR11MB63928CC05906F27FB10D74D0FD322@CY5PR11MB6392.namprd11.prod.outlook.com>
	<ko5nio3iyt7fonvwqqgs6gddzyzlad7qm5v6zo72h753jmitwy@llz3bghejzig>
	<175863756865.1038.1316404745203068811.pgcf@coridan.postgresql.org>
	<3xsqvatpnbb7wd3vdxdygzh6s2bbxn3ulmarli67x5u7hekvei@3it2vxprjglu>
	<SJ4PPFB817783263D506AAE4C480F5B2866DB1DA@SJ4PPFB81778326.namprd15.prod.outlook.com>
	<SJ4PPFB8177832600E5AAE4214587DC3F7BDBE8A@SJ4PPFB81778326.namprd15.prod.outlook.com>

On 15.10.25 17:58, Srirama Kucherlapati wrote:
> Hi Andres and team,
> 
> Please find attached the patch addressing Meson build changes for AIX.
> 
> We tried to build using below meson
> 
> ## meson -v
> 
> 1.6.1
> 
> For the following files, the shared_module() function requires 
> specifying the
> 
> appropriate symbol import file, which is provided via the aix_link_args
> 
> variable:
> 
> src/backend/jit/llvm/meson.build
> 
> src/backend/replication/libpqwalreceiver/meson.build
> 
> src/backend/replication/pgoutput/meson.build
> 
> src/backend/snowball/meson.build
> 
> src/backend/utils/mb/conversion_procs/meson.build
> 
> Additionally, with the latest version of Meson, when building static 
> libraries,
> 
> the library file name must be suffixed with “_static” in the following 
> files:
> 
> src/interfaces/ecpg/compatlib/meson.build
> 
> src/interfaces/ecpg/ecpglib/meson.build
> 
> src/interfaces/ecpg/pgtypeslib/meson.build
> 
> src/interfaces/libpq/meson.build

It is not going to be acceptable to sprinkle this much AIX-specific code 
around.  Those changes to the meson.build files were not needed before 
AIX support was removed.  You should find a way to do it without these 
changes.

The changes in src/include/port/aix.h in your patch don't make sense. 
This indicates that you are doing something wrong in your local builds. 
Please fix that.

view thread (73+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: AIX support
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox