Horton
Menu

Re: uintptr_t for Datum

Peter Eisentraut <peter_e@gmx.net>

View thread
From: Peter Eisentraut <peter_e@gmx.net>
To: Bruce Momjian <bruce@momjian.us>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Magnus Hagander <magnus@hagander.net>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2010-01-01T18:58:20Z
Lists: pgsql-hackers 
On tor, 2009-12-31 at 13:44 -0500, Bruce Momjian wrote:
> Tom Lane wrote:
> > Bruce Momjian <bruce@momjian.us> writes:
> > > Tom Lane wrote:
> > >> BTW, it looks like the patch is showing a manual change to
> > >> pg_config.h.in.  Don't do that.  Run autoheader.
> > 
> > > I wasn't aware autoheader existed.  Is that new or has it alwasy been
> > > part of autoconf?
> > 
> > It's always been there, or at least for many years.  pg_config.h.in
> > really ought to be thought of the same as configure: you don't edit
> > it, you just generate it.
> 
> Well, that's pretty confusing considering it has a .in suffix, just like
> configure.in, which we do edit, but I get your point.

I realize it's easy to miss, but the first line of pg_config.h.in tells
that it is generated.