Re: typedefs for indent - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: typedefs for indent
Date
Msg-id 200903270223.n2R2NlN09673@momjian.us
Whole thread Raw
In response to Re: typedefs for indent  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-hackers
Andrew Dunstan wrote:
> This URL <http://www.pgbuildfarm.org/cgi-bin/typedefs.pl> gives a 
> typedef list that is (currently) the combined result from three fairly 
> different buildfarm members:
> 
> dungbeetle | 2009-03-22 06:44:01
> brown_bat | 2009-03-21 13:00:58
> dawn_bat | 2009-03-21 14:23:40
> 
> These are respectively my Linux, Cygwin and MinGW buildfarm members.
> 
> I don't have a BSD machine of any flavor to test on, and I don't know 
> how to extract the typedefs on OSX.
> 
> Anyone running a buildfarm member should be able to do this and add to 
> the results, if they are up to date with release 3.2. I have my linux 
> crontab set up to do one typedefs run on the HEAD branch each day.

[ Discussion deleted.]

Andrew, this is disappointing news.  When you talked about generating an
typedef list from the buildfarm, you were saying how great it would be
--- now a year later you post:
It'd be nice to get that dealt with before we run pg_indent, but it'snot like we'd be any worse off than before if we
don't.In any case it'ssurely no blocker for 8.4beta.
 

We can't have the system-supplied typedef list changing from release to
release because that affects the indenting from release to release,
which affects backpatching and other stuff.  And even if you get a more
complete list then we have used in the past, what are the odds you are
going to supply a typedef that is a typedef on some operating system
that matches an identifier in our code that _isn't_ used as a typedef by
us?

We only have a few weeks until I have to run pgindent so I would like
this resolved one way or another soon.  Unless I hear otherwise I assume
we are going to just use the an updated list of our defined typedefs
that gets generated from our code, which includes my BSD typedefs.

One other approach would be to include in pg_indent a hard-coded list of
non-BSD system-defined typedefs that we reference from our code.  One
way to find those would be to run pg_indent with and without Andrew's
list of typedefs and see how the formatting changes.

Or just use a Linux list of system typedefs from now on.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +


pgsql-hackers by date:

Previous
From: Greg Stark
Date:
Subject: Re: 8.4 release notes proof reading 1/2
Next
From: Bruce Momjian
Date:
Subject: Re: 8.4 release notes proof reading 1/2