From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | Jakob Egger <jakob(at)eggerapps(dot)at> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Use Python "Limited API" in PL/Python |
Date: | 2025-01-15 22:20:58 |
Message-ID: | 3266c8e1-478e-47c0-8405-716687489f68@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 15.01.25 12:28, Jakob Egger wrote:
> On 14.01.2025, at 16:51, Jakob Egger <jakob(at)eggerapps(dot)at> wrote:
>> I've tried to create a patch with this change. I'm attaching it to
>> this message so that cfbot picks it up. (I was unable to reproduce the
>> issue locally)
>
> Apologies, please disregard my last patch. It does not work.
>
> It looks like meson just can't build libraries that link with Python
> that use the limited API.
>
> My understanding of Meson is limited, so I am unsure how to best work
> around this limitation.
>
> I've spent two days trying to figure it out; I'm at the end of my wits.
Thanks for checking. I've also been poking at this, without success.
What we could do is just not activate the limited API on Windows
(actually MSVC, since it works on MinGW). If someone later finds a way
to make it work on Windows/MSVC, then it's easy to change.
So here is an updated patch set with that change. It should pass cfbot
like this.
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Remove-obsolete-Python-version-check.patch | text/plain | 1.2 KB |
v2-0002-Prepare-for-Python-Limited-API-in-PL-Python.patch | text/plain | 15.7 KB |
v2-0003-Activate-Python-Limited-API-in-PL-Python.patch | text/plain | 1.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Jeff Davis | 2025-01-15 22:34:52 | Re: Allow ILIKE forward matching to use btree index |
Previous Message | Peter Eisentraut | 2025-01-15 22:14:24 | Re: Index AM API cleanup |