Re: Use Python "Limited API" in PL/Python

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

In response to

Browse pgsql-hackers by date

  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