| 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: | Whole Thread | Raw Message | 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 |
| 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 |