Re: Initial refactoring of plperl.c - updated - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Initial refactoring of plperl.c - updated
Date
Msg-id 25644.1263104173@sss.pgh.pa.us
Whole thread Raw
In response to Re: Initial refactoring of plperl.c - updated  (Tim Bunce <Tim.Bunce@pobox.com>)
Responses Re: Initial refactoring of plperl.c - updated  (Tim Bunce <Tim.Bunce@pobox.com>)
List pgsql-hackers
Tim Bunce <Tim.Bunce@pobox.com> writes:
> On Sat, Jan 09, 2010 at 11:16:18PM +0200, Peter Eisentraut wrote:
>> What's the reason for the temp file here?

> Defensive. If the text2macro.pl program fails/dies then you'd be left
> with a broken output file with a newer timestamp, so the next make
> wouldn't rerun text2macro.pl.

Doesn't make forcibly remove the target file if the command fails?

[ rummages in manual... ]  It does if you specify .DELETE_ON_ERROR,
which we do (in Makefile.global); so this bit of complication is
a waste.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Oleg Bartunov
Date:
Subject: Re: Congrats Alvaro!
Next
From: Tom Lane
Date:
Subject: Re: win32 socket definition