Re: AIX support

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Aditya Kamath <Aditya(dot)Kamath1(at)ibm(dot)com>
Cc: Srirama Kucherlapati <sriram(dot)rk(at)in(dot)ibm(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Noah Misch <noah(at)leadboat(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Robert Haas <robertmhaas(at)gmail(dot)com>, "peter(at)eisentraut(dot)org" <peter(at)eisentraut(dot)org>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "hlinnaka(at)iki(dot)fi" <hlinnaka(at)iki(dot)fi>, "tristan(at)partin(dot)io" <tristan(at)partin(dot)io>, "postgres-ibm-aix(at)wwpdl(dot)vnet(dot)ibm(dot)com" <postgres-ibm-aix(at)wwpdl(dot)vnet(dot)ibm(dot)com>
Subject: Re: AIX support
Date: 2026-02-16 15:26:19
Message-ID: 1122960.1771255579@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Aditya Kamath <Aditya(dot)Kamath1(at)ibm(dot)com> writes:
> +# -blibpath must contain ALL directories where we should look for libraries
> +libpath := $(shell echo $(subst -L,:,$(filter -L/%,$(LDFLAGS))) | sed -e's/ //g'):/usr/lib:/lib
> +
> +# when building with gcc, need to make sure that libgcc can be found
> +ifeq ($(GCC), yes)
> +libpath := $(libpath):$(dir $(shell gcc -print-libgcc-file-name))
> +endif
> +
> +rpath = -Wl,-blibpath:'$(rpathdir)$(libpath)’

> These lines are no longer required, and we have removed it in v12 patch as well. And this is the root cause of GNU tests failing.

Hmm, I'm not sure how "these lines are not required" squares with
"the tests fail after we remove them". More generally, we do need
rpath support for many installation scenarios. So even if you can
make it work for your own scenario without that, I think the
answer is "you can use --disable-rpath" not "we should remove rpath
support for everybody".

> After removing the lib path and path from Makefile.aix we get the below library search paths which is what we need to run test cases or run binaries,

> ***Import File Strings***
> INDEX PATH BASE MEMBER
> 0 ../../../src/port:../../../src/common:../../../src/fe_utils:../../../src/interfaces/libpq:/opt/freeware/lib:/opt/freeware/lib/gcc/powerpc-ibm-aix7.3.0.0/13/ppc64:/opt/freeware/lib/gcc/powerpc-ibm-aix7.3.0.0/13/../../../ppc64:/opt/freeware/lib/gcc/powerpc-ibm-aix7.3.0.0/13:/opt/freeware/lib/gcc/powerpc-ibm-aix7.3.0.0/13/../../..:/usr/lib:/lib

You should be quite scared of that result, because the relative paths
will only work in a build tree not for an installed server executable.

My recollection is that I put those lines back after discovering
that "make install; make installcheck" didn't work without 'em.
That was on the GCC farm's machine, so there may also be an
AIX 7.2 vs 7.3 difference involved here, and/or some difference
in the configuration of the gcc compiler.

I intend to test the final patch version on cfarm119 before
committing, but it has been down for a couple days, no doubt
because of OSUOSL's ongoing data center move.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Anthonin Bonnefoy 2026-02-16 15:30:45 Re: Fix uninitialized xl_running_xacts padding
Previous Message Bruce Momjian 2026-02-16 15:16:56 Re: Support logical replication of DDLs