Re: pgsql: Improve support for building PGXS modules with VPATH. - Mailing list pgsql-committers

From Cédric Villemain
Subject Re: pgsql: Improve support for building PGXS modules with VPATH.
Date
Msg-id 201307012313.59438.cedric@2ndquadrant.com
Whole thread Raw
In response to Re: pgsql: Improve support for building PGXS modules with VPATH.  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: pgsql: Improve support for building PGXS modules with VPATH.  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-committers
Le lundi 1 juillet 2013 21:22:55, Andrew Dunstan a écrit :
> On 07/01/2013 02:40 PM, Andrew Dunstan wrote:
> > On 07/01/2013 01:39 PM, Alvaro Herrera wrote:
> >> Andrew Dunstan wrote:
> >>> Improve support for building PGXS modules with VPATH.
> >>>
> >>> A VPATH build will be performed when the module's make file path is not
> >>> the current directory or when USE_VPATH is set.
> >>
> >> So what's the USE_VPATH thing for?  I mean, can't you just use VPATH
> >> instead?
> >
> > Well, I was distinguishing between things set by the Makefile and
> > things set outisde, but you're probably right. I'll test it and tweak.
>
> Actually, when I try to get rid of it I get issues with srcdir being
> recursively defined.

As for VPATH, the srcdir was previously hard-coded, the patch just improve
most of the use cases by setting a more appropriate value..

> And in fact, VPATH is defined by Makefile.global when the Postgres build
> is a VPATH build, and unrelated to the module's build setup.

pgxs.mk already (I mean before the last commit) set VPATH, the patch around
that just try to fix it to be correct instead of hardcoded to NULL.. What's in
Makefile.global is not be affected.

--
Cédric Villemain +33 (0)6 20 30 22 52
http://2ndQuadrant.fr/
PostgreSQL: Support 24x7 - Développement, Expertise et Formation

Attachment

pgsql-committers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: pgsql: Add a convenience routine makeFuncCall to reduce duplication.
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Fix regression test make dependencies