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

From Tim Bunce
Subject Re: Initial refactoring of plperl.c - updated
Date
Msg-id 20100110000333.GF2481@timac.local
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
List pgsql-hackers
On Sat, Jan 09, 2010 at 11:49:22PM +0000, Tim Bunce wrote:
> On Sat, Jan 09, 2010 at 11:16:18PM +0200, Peter Eisentraut wrote:
> > On fre, 2010-01-08 at 12:46 +0000, Tim Bunce wrote:
> > > *** 45,50 ****
> > > --- 45,55 ----
> > >   
> > >   include $(top_srcdir)/src/Makefile.shlib
> > >   
> > > + plperl.o: perlchunks.h
> > > + 
> > > + perlchunks.h: plc_*.pl
> > > +       $(PERL) text2macro.pl --strip='^(\#.*|\s*)$$' plc_*.pl >
> > > perlchunks.htmp
> > > +       mv perlchunks.htmp perlchunks.h
> > >   
> > >   all: all-lib
> > 
> > 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.

An alternative would be for text2macro.pl to write to a temp file and
rename at the end.

Tim.


pgsql-hackers by date:

Previous
From: Tim Bunce
Date:
Subject: Re: Initial refactoring of plperl.c - updated
Next
From: Andrew Dunstan
Date:
Subject: Re: Feature patch 1 for plperl [PATCH]