Re: POC: Sharing record typmods between backends

On 2017-09-14 23:29:05 -0400, Tom Lane wrote:
> Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> writes:
> > On Fri, Sep 15, 2017 at 3:03 PM, Andres Freund <andres(at)anarazel(dot)de> wrote:
> >> - added typedefs to typedefs.list
>
> > Should I do this manually with future patches?

I think there's sort of a circuit split on that one. Robert and I do
regularly, most others don't.

> FWIW, I'm not on board with that. I think the version of typedefs.list
> in the tree should reflect the last official pgindent run.

Why? I see pretty much no upside to that. You can't reindent anyway, due
to unindented changes. You can get the used typedefs.list trivially from
git.

> There's also a problem that it only works well if *every* committer
> faithfully updates typedefs.list, which isn't going to happen.
>
> For local pgindent'ing, I pull down
>
> https://buildfarm.postgresql.org/cgi-bin/typedefs.pl
>
> and then add any typedefs created by the patch I'm working on to that.
> But I don't put the result into the commit. Maybe we need a bit better
> documentation and/or tool support for using an unofficial typedef list.

That's a mighty manual process - I want to be able to reindent files,
especially new ones where it's still reasonably possible, without having
to download files, then move changes out of the way, so I can rebase,
...

Greetings,

Andres Freund