Re: Bugfix and new feature for PGXS

From: Cédric Villemain <cedric(at)2ndquadrant(dot)com>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org, Josh Berkus <josh(at)agliodbs(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Craig Ringer <craig(at)2ndquadrant(dot)com>
Subject: Re: Bugfix and new feature for PGXS
Date: 2013-09-30 06:50:38
Message-ID: 41858909.8rl4vTGUkV@obelix
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Le lundi 30 septembre 2013 00:10:09 Andrew Dunstan a écrit :
> On 09/29/2013 07:09 PM, Andrew Dunstan wrote:
> > On 09/03/2013 04:04 AM, Cédric Villemain wrote:
> >>> Simple one, attached.
> >>> I didn't document USE_VPATH, not sure how to explain that clearly.
> >>
> >> Just a remember that the doc is written and is waiting to be
> >> commited.
> >>
> >> There is also an issue spoted by Christoph with the installdirs
> >> prerequisite,
> >> the attached patch fix that.
> >
> > I applied this one line version of the patch, which seemed more
> > elegant than the later one supplied.
> >
> > I backpatched that and the rest of the VPATH changes for extensiuons
> > to 9.1 where we first provided for extensions, so people can have a
> > reasonably uniform build system for their extensions.
>
> I have reverted this in the 9.2 and 9.1 branches until I can work out
> why it's breaking the buildfarm. 9.3 seems to be fine.

Yes, please take the longer but better patch following the small one.
There are eveidences that it fixes compilation to always build
installdirs first.
Previously installdirs may be build after copying files, so it fails.
Chritstoph authored this good patch.

I'm sorry not to have been clear on that.
--
Cédric Villemain +33 (0)6 20 30 22 52
http://2ndQuadrant.fr/
PostgreSQL: Support 24x7 - Développement, Expertise et Formation

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message David Rowley 2013-09-30 09:10:56 Re: appendStringInfo vs appendStringInfoString
Previous Message Daniel Farina 2013-09-30 05:58:58 Re: pg_stat_statements: calls under-estimation propagation