Re: error out when building pg_xlogdump with pgxs

From: Cédric Villemain <cedric(at)2ndquadrant(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Cc: Andres Freund <andres(at)2ndquadrant(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Subject: Re: error out when building pg_xlogdump with pgxs
Date: 2013-08-29 09:49:00
Message-ID: 1567138.mGljfHUSvy@obelix
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Le mercredi 28 août 2013 00:12:22 Andres Freund a écrit :
> Hi Alvaro,
>
> On 2013-08-27 14:47:49 -0400, Alvaro Herrera wrote:
> > Andres Freund wrote:
> > > pg_xlogdump cannot properly be built with pgxs since it needs a
> > > sourcetree around. That already has confused some users...
> > >
> > > How about the attached patch which will tell it's not supported instead
> > > of an ominous build error about files that have no rules?
> >
> > Hmm, the other option is to ignore USE_PGXS completely and build
> > assuming the source tree is always present. That way, if you build the
> > whole contrib subdir with USE_PGXS=1, you will end up with all modules
> > being built, not stop in the middle with an error. This seems more
> > useful to me. We could just add a comment that USE_PGXS is ignored.
>
> What's the point in doing USE_PGXS builds with a full and configured
> source present? The only thing I can think of is testing that pgxs
> builds are working. In that case it doesn't seem helpful to fake
> something into working which is then going to fail for real USE_PGXS
> builds (where the original sourcetree won't be at that location
> anymore).

I had the same idea when Peter wished to remove PGXS from the contrib shiped
with postgreSQL.

I've been convinced that if we want to apply testing on pgxs makefile then we
need something dedicated. Not abusing the current options.

I'm in favor of removing PGXS from all contrib makefile, not only this one.

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2013-08-29 09:52:36 Re: error out when building pg_xlogdump with pgxs
Previous Message Pavel Stehule 2013-08-29 05:48:45 Re: Improving avg performance for numeric