From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Peter Eisentraut <peter(at)eisentraut(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Nathan Bossart <nathandbossart(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Why does pgindent's README say to download typedefs.list from the buildfarm? |
Date: | 2024-04-24 10:37:00 |
Message-ID: | 0d751dc8-0dd0-4335-a429-f43ad1e1db2f@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2024-04-24 We 06:12, Peter Eisentraut wrote:
> On 22.04.24 22:28, Tom Lane wrote:
>> Nathan Bossart<nathandbossart(at)gmail(dot)com> writes:
>>> On Mon, Apr 22, 2024 at 04:08:08PM -0400, Tom Lane wrote:
>>>> I think the actual plan now is that we'll sync the in-tree copy
>>>> with the buildfarm's results (and then do a tree-wide pgindent)
>>>> every so often, probably shortly before beta every year.
>>> Okay. Is this just to resolve the delta between the manual updates
>>> and a
>>> clean autogenerated copy every once in a while?
>> The main reason there's a delta is that people don't manage to
>> maintain the in-tree copy perfectly (at least, they certainly
>> haven't done so for this past year). So we need to do that
>> to clean up every now and then.
>>
>> A secondary reason is that the set of typedefs we absorb from
>> system include files changes over time.
>
> Is the code to extract typedefs available somewhere independent of the
> buildfarm? It would be useful sometimes to be able to run this
> locally, like before and after some patch, to keep the in-tree
> typedefs list updated.
>
>
>
There's been talk about it but I don't think anyone's done it. I'd be
more than happy if the buildfarm client could just call something in the
core repo (c.f. src/test/perl/Postgres/Test/AdjustUpgrade.pm).
Regarding testing with your patch, some years ago I wrote this blog
post:
<http://adpgtech.blogspot.com/2015/05/running-pgindent-on-non-core-code-or.html>
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2024-04-24 10:41:06 | Re: Remove unnecessary code rom be_lo_put() |
Previous Message | Yugo NAGATA | 2024-04-24 10:34:22 | Re: Rename libpq trace internal functions |