Re: [PATCH] Remove useless USE_PGXS support in contrib - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: [PATCH] Remove useless USE_PGXS support in contrib
Date
Msg-id 51C212FE.3040305@dunslane.net
Whole thread Raw
In response to Re: [PATCH] Remove useless USE_PGXS support in contrib  (Dimitri Fontaine <dimitri@2ndQuadrant.fr>)
Responses Re: [PATCH] Remove useless USE_PGXS support in contrib  (Dimitri Fontaine <dimitri@2ndQuadrant.fr>)
Re: [PATCH] Remove useless USE_PGXS support in contrib  (Cédric Villemain <cedric@2ndquadrant.com>)
List pgsql-hackers
On 06/19/2013 03:52 PM, Dimitri Fontaine wrote:
> Peter Eisentraut <peter_e@gmx.net> writes:
>> We could do something like
>>
>> PG_CONFIG = fake_intree_pg_config
>> PGXS := $(shell $(PG_CONFIG) --pgxs)
>> include $(PGXS)
> There's something to that idea. Of course we would need to offer a
> comment about the PG_CONFIG game and propose something else for real
> world extensions (PG_CONFIG ?= pg_config).
>
>> where fake_intree_pg_config is a purpose-built shell script that points
>> to the right places inside the source tree.
> If that works, that gets my preference over removing PGXS support in
> contrib modules. Setting an example is important, in-tree build is not
> a useful example for anyone but contributors to core.
>


Not true - you're forgetting there is no pgxs for MSVC builds.

If we're going to enable building of contrib modules using pgxs but 
without an install we will make targets for that, and buildfarm support.

cheers

andrew



pgsql-hackers by date:

Previous
From: Kevin Grittner
Date:
Subject: FP precision
Next
From: "Joshua D. Drake"
Date:
Subject: Re: Change authentication error message (patch)