| From: | Aditya Kamath <Aditya(dot)Kamath1(at)ibm(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | 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>, Srirama Kucherlapati <sriram(dot)rk(at)in(dot)ibm(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-03 15:23:54 |
| Message-ID: | LV8PR15MB64884402D55FC9207CB3E993D69BA@LV8PR15MB6488.namprd15.prod.outlook.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Tom and community members,
Please find attached the patch (See: v4-0001-Add-AIX-support-for-Postgresql-using-meson-build-.patch)
You will need to git apply your patch and then this patch.
I had missed out a few things like the die message change in the .pl file in the v3 of this patch.
Also, the export OBJECT_MODE=64 flag is tentatively only needed.
I maintain meson in AIX and have raised a PR (link below) to eliminate the need to do the same. Once merged will update meson in AIX and we will no longer need to export the same.
https://github.com/mesonbuild/meson/pull/15516
So this puts AIX in sync with other targets and we can also now just do meson setup build && meson compile -C build to build the project.
Have a nice day ahead.
Thanks and regards,
Aditya.
From: Aditya Kamath <Aditya(dot)Kamath1(at)ibm(dot)com>
Date: Monday, 2 February 2026 at 11:40 PM
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: 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>, Srirama Kucherlapati <sriram(dot)rk(at)in(dot)ibm(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: [EXTERNAL] Re: AIX support
Hi Tom,
Please find attached the patch (See: v3-0001-Add-AIX-support-for-Postgresql-using-meson-build-.patch)
You will have to apply your patch https://www.postgresql.org/message-id/attachment/189255/v1-0001-Decouple-our-alignment-assumptions-about-int64-an.patch and then git apply v3-0001-Add-AIX-support-for-Postgresql-using-meson-build-.patch and then follow the meson build instructions in the commit message.
> Pushed at 0c9f46c42.
Thank you for the push. We have updated our branch. And this patch is made after the update.
>Please, don't squash all of the changes under discussion into a single
>patch. We aren't likely to commit it like that, and merging different
>topics into one patch just makes it harder to review everything.
>In this case, it'd have been much better if you kept my changes from
>https://urldefense.proofpoint.com/v2/url?u=https-3A__www.postgresql.org_message-2Did_1694571.1769736080-40sss.pgh.pa.us&d=DwIFAg&c=BSDicqBQBDjDI9RkVyTcHQ&r=f-oUQ8ByG1nZ71OI9p76qywCPh7mxzU69hBYnkP9Nis&m=dyY-gcWS2Xjotl8-Qex9QwI6MDbnbGcKvghelG6GrW7SVQL3kExDhCzFif0sWkTA&s=Nbdnq5St72HOehkkV5kU2jH_Ofyg6Z7tKt846xg-plg&e=
>separate from the rest, so that it'd be easier to compare your work
>to the previous version 0001-Support-for-AIX.pg19.v11.patch.
>Also, please pretty please use systematic naming including a version
>number for new patches. It's far from obvious that
>0001-Add-AIX-support-for-Postgresql-using-meson-build-too.patch is
>newer than 0001-Support-for-AIX.pg19.v11.patch. (If you're making
>the patch files with git format-patch, use "-v N".)
I have taken care of the same. Please see now and let me know.
Rest of the comments same as previous email comments.
Have a nice day.
Regards,
Aditya.
| Attachment | Content-Type | Size |
|---|---|---|
| v4-0001-Add-AIX-support-for-Postgresql-using-meson-build-.patch | application/octet-stream | 19.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2026-02-03 15:28:49 | Re: Pasword expiration warning |
| Previous Message | Andres Freund | 2026-02-03 15:14:11 | Re: Remove freelist reference in buf_init.c |