Re: meson PGXS compatibility

From: Andres Freund <andres(at)anarazel(dot)de>
To: Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: meson PGXS compatibility
Date: 2022-12-01 20:17:51
Message-ID: 20221201201751.4y6g57yomprq3rlw@awork3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2022-12-01 08:43:19 +0100, Peter Eisentraut wrote:
> On 13.10.22 07:23, Andres Freund wrote:
> > > > 0005: meson: Add PGXS compatibility
> > > >
> > > > The actual meson PGXS compatibility. Plenty more replacements to do, but
> > > > suffices to build common extensions on a few platforms.
> > > >
> > > > What level of completeness do we want to require here?
> > >
> > > I have tried this with a few extensions. Seems to work alright. I don't
> > > think we need to overthink this. The way it's set up, if someone needs
> > > additional variables set, they can easily be added.
> >
> > Yea, I am happy enough with it now that the bulk is out of src/meson.build and
> > strip isn't set to an outright wrong value.
>
> How are you planning to proceed with this? I thought it was ready, but it
> hasn't moved in a while.

I basically was hoping for a review of the prerequisite patches I posted at
[1], particularly 0003 (different approach than before, comparatively large).

Here's an updated version of the patches. There was a stupid copy-paste bug in
the prior version of the 0003 / export_dynamic patch.

I'll push 0001, 0002 shortly. I don't think 0002 is the most elegant approach,
but it's not awful. I'd still like some review for 0003, but will try to push
it in a few days if that's not forthcoming.

Greetings,

Andres Freund

[1] https://www.postgresql.org/message-id/20221013051648.ufz7ud2b5tioctyt%40awork3.anarazel.de

Attachment Content-Type Size
v4-0001-autoconf-Unify-CFLAGS_SSE42-and-CFLAGS_ARMV8_CRC3.patch text/x-diff 9.7 KB
v4-0002-autoconf-Don-t-AC_SUBST-LD-in-configure.patch text/x-diff 3.2 KB
v4-0003-autoconf-Move-export_dynamic-determination-to-con.patch text/x-diff 25.6 KB
v4-0004-meson-Add-PGXS-compatibility.patch text/x-diff 11.7 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2022-12-01 20:49:42 Re: Error-safe user functions
Previous Message Robert Haas 2022-12-01 20:12:09 Re: Error-safe user functions