Re: AIX support

From: AIX PG user <postgres-ibm-aix(at)wwpdl(dot)vnet(dot)ibm(dot)com>
To: pgsql-hackers(at)lists(dot)postgresql(dot)org
Cc: AIX PG user <postgres-ibm-aix(at)wwpdl(dot)vnet(dot)ibm(dot)com>
Subject: Re: AIX support
Date: 2025-11-10 17:00:13
Message-ID: 176279401378.2081919.12877701948713975661.pgcf@coridan.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Peter,
>> 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 change was necessary to ensure that the import file used during the static
PostgreSQL library build begins with the required #! directive and includes the
correct symbol references from the object files. This structure is essential
for compatibility with AIX's linking behaviour, which relies on explicitly
defined symbol import files during static linking. Without this adjustment, the
build process may fail or result in incomplete symbol resolution, particularly
on AIX systems. By aligning the import file format and symbol inclusion with
platform-specific requirements.

We referred to previously removed code that handled similar logic and found it
to be effective. Based on this, we plan to revert the changes in the affected
files and proceed with a more robust solution.

https://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=0b16bb8776bb834eb1ef8204ca95dd7667ab948b#patch11
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

Regarding the archiving of shared and static libraries, the latest version of
Meson raises a conflict as both targets have the same name. This issue
arises because, on AIX, both shared and static libraries are named as same
archive, which leads to naming collisions during the build process. We
would still need the below changes.

src/interfaces/ecpg/compatlib/meson.build
src/interfaces/ecpg/ecpglib/meson.build
ecpglib_st = static_library(libname,
src/interfaces/ecpg/pgtypeslib/meson.build
ecpg_pgtypes_st = static_library(libname,
src/interfaces/libpq/meson.build
libpq_st = static_library(libname

>> 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.

We had to do these changes in the src/include/c.h,
As the MACROS, are getting undefined by the below sys header files.

We made changes to src/include/c.h because certain macros were being
undefined by the system header files listed below. To address this issue,
we introduced updates in aix.h.
+#ifndef PGDLLIMPORT +#define PGDLLIMPORT

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

We are continuing to analyze the impact of these changes and will refine
the solution as needed. Once finalized, we will submit the updated changes accordingly.

-Sriram.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Yugo Nagata 2025-11-10 17:07:01 Make PQgetResult() not return NULL on out-of-memory error
Previous Message Rahila Syed 2025-11-10 16:50:19 Re: Extend injection_points_attach() to accept a user-defined function