Re: apply pragma system_header to python headers

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Peter Eisentraut <peter(at)eisentraut(dot)org>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: apply pragma system_header to python headers
Date: 2023-12-25 17:36:50
Message-ID: 1880816.1703525810@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I wrote:
> Probably a better way is to invent a separate header "plpython_system.h"
> that just includes the Python headers, to scope the pragma precisely.
> (I guess it could have the fixup #defines we're wrapping those headers
> in, too.)

> The same idea would work in plperl.

After updating one of my test machines to Fedora 39, I'm seeing these
Python warnings too. So here's a fleshed-out patch proposal for doing
it like that.

regards, tom lane

Attachment Content-Type Size
v2-apply-system_header-pragma.patch text/x-diff 16.6 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2023-12-25 17:48:17 No LINGUAS file yet for pg_combinebackup
Previous Message Michail Nikolaev 2023-12-25 14:12:41 Re: Revisiting {CREATE INDEX, REINDEX} CONCURRENTLY improvements