Re: [COMMITTERS] pgsql: Remove too-smart-for-its-own-good optimization of not overwriting - Mailing list pgsql-hackers

From Greg Stark
Subject Re: [COMMITTERS] pgsql: Remove too-smart-for-its-own-good optimization of not overwriting
Date
Msg-id 407d949e1001041953n29596b6kcf41f753cc078e6b@mail.gmail.com
Whole thread Raw
In response to Re: [COMMITTERS] pgsql: Remove too-smart-for-its-own-good optimization of not overwriting  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: [COMMITTERS] pgsql: Remove too-smart-for-its-own-good optimization of not overwriting
Re: Re: [COMMITTERS] pgsql: Remove too-smart-for-its-own-good optimization of not overwriting
List pgsql-hackers
On Tue, Jan 5, 2010 at 3:42 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> On Mon, Jan 4, 2010 at 9:34 PM, Tom Lane <tgl@postgresql.org> wrote:
>> Log Message:
>> -----------
>> Remove too-smart-for-its-own-good optimization of not overwriting the output
>> files when they haven't changed.  This confuses make because the build fails
>> to update the file timestamps, and so it keeps on doing the action over again.
>
> This doesn't seem like a good idea.  Not rebuilding the output files
> also saves recompiling the things that depend on them.  For the BKI
> files thast doesn't matter much, but for schemapg.h it might be
> significant.  Certainly, if we move to generating more header files
> this way, it WILL be significant.  If running the script is cheap (and
> it should be), it's better to take that hit rather than recompiling a
> whole mess of .c files unnecessarily.


I think there's a trick to cover this case but I don't recall what it is.

Does generating a stamp file help? If you had a rule saying to trigger
generating the output files because the stamp file is out of date
which might or might not touch the .h file which would trigger more
files to be rebuilt then everything should work.... except I fear this
leads us back to the "make rule which generates two files" problem...

--
greg


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [COMMITTERS] pgsql: Remove too-smart-for-its-own-good optimization of not overwriting
Next
From: Robert Haas
Date:
Subject: Re: [COMMITTERS] pgsql: Remove too-smart-for-its-own-good optimization of not overwriting