From: Tom Lane <tgl@sss.pgh.pa.us>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: John Naylor <jcnaylor@gmail.com>
Cc: Stas Kelvich <s.kelvich@postgrespro.ru>
Cc: Andreas Karlsson <andreas@proxel.se>
Cc: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Cc: David Fetter <david@fetter.org>
Subject: Re: unused_oids script is broken with bsd sed
Date: Thu, 03 May 2018 16:11:25 -0400
Message-ID: <18604.1525378285@sss.pgh.pa.us> (raw)
In-Reply-To: <20180503195642.otwsjbjdtzdemumf@alvherre.pgsql>
References: <20180503195642.otwsjbjdtzdemumf@alvherre.pgsql>
Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> Tom Lane wrote:
>> However, RenameTempFile is also used by Gen_fmgrtab.pl, and having the
>> same sort of no-touch semantics for fmgroids.h and friends would have some
>> additional fallout. The makefiles would think they have to keep
>> re-running Gen_fmgrtab.pl if fmgroids.h is older than the mod time on any
>> input file, and that's certainly no good. We can fix that by inventing a
>> stamp file for the Gen_fmgrtab.pl run, analogous to bki-stamp for the
>> genbki.pl run, but that means changes in the makefiles that go a bit
>> beyond the realm of triviality.
> Sounds OK to me -- a stamp file is already established technique, so it
> shouldn't go *too much* beyond triviality.
Yeah, what I'm envisioning is to change the makefile rules around these
files to look as much as possible like the ones around the BKI files,
which are (we hope) already debugged. So it doesn't seem like a high
risk change ... at least so far as the makefiles are concerned.
> I do note that
> msvc/Solution.pm runs Gen_fmgrtab.pl, but it shouldn't require any
> changes anyhow.
Hmm. Actually, given the IsNewer checks there, it looks like Solution.pm
is basically hand-rolling makefile-like dependency checking, which means
it would be fooled by no-touch updates in the same way as make is, causing
rebuilds to do unnecessary work. We could live with that for awhile
maybe, but ultimately Solution.pm would need to be fixed to use a stamp
file like the makefile logic.
I could take a stab at that, but I don't have any way to test it myself.
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: tgl@sss.pgh.pa.us, alvherre@2ndquadrant.com, robertmhaas@gmail.com, jcnaylor@gmail.com, s.kelvich@postgrespro.ru, andreas@proxel.se, andrew.dunstan@2ndquadrant.com, david@fetter.org
Subject: Re: unused_oids script is broken with bsd sed
In-Reply-To: <18604.1525378285@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox