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