From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Tristan Partin <tristan(at)neon(dot)tech> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Clean up find_typedefs and add support for Mac |
Date: | 2023-12-13 20:35:40 |
Message-ID: | 02fe2980-e9e0-21fb-1a85-f992a591788c@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2023-12-12 Tu 18:02, Tom Lane wrote:
> "Tristan Partin" <tristan(at)neon(dot)tech> writes:
>> The big patch here is adding support for Mac. objdump -W doesn't work on
>> Mac. So, I used dsymutil and dwarfdump to achieve the same result.
> We should probably nuke the current version of src/tools/find_typedef
> altogether in favor of copying the current buildfarm code for that.
> We know that the buildfarm's version works, whereas I'm not real sure
> that src/tools/find_typedef is being used in anger by anyone. Also,
> we're long past the point where developers can avoid having Perl
> installed.
>
> Ideally, the buildfarm client would then start to use find_typedef
> from the tree rather than have its own copy, both to reduce
> duplication and to ensure that the in-tree copy keeps working.
>
>
+1. I'd be more than happy to be rid of maintaining the code. We already
performed a rather more complicated operation along these lines with the
PostgreSQL::Test::AdjustUpgrade stuff.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Imseih (AWS), Sami | 2023-12-13 20:42:45 | [BUG] autovacuum may skip tables when session_authorization/role is set on database |
Previous Message | Thomas Munro | 2023-12-13 19:58:51 | Re: Windows default locale vs initdb |