Thread: Re: [HACKERS] pgxs/windows

Re: [HACKERS] pgxs/windows

From
"Rocco Altier"
Date:
If I understand correctly, atleast for the AIX case, the reason that
Makefile.aix is referencing DESTDIR is for the PGXS case.

I thought that PGXS was supposed to be able to be run after things were
installed, and you might not have the compile tree anymore.  Basically,
if you wanted to build some extensions on top of a precompiled set of
binaries.

If we take the DESTDIR out of here, where should we get the POSTGRES_IMP
file from?

    -rocco

> -----Original Message-----
> From: pgsql-patches-owner@postgresql.org
> [mailto:pgsql-patches-owner@postgresql.org] On Behalf Of Tom Lane
> Sent: Thursday, January 19, 2006 3:54 PM
> To: Bruce Momjian
> Cc: Andrew Dunstan; Magnus Hagander; PostgreSQL-patches
> Subject: Re: [PATCHES] [HACKERS] pgxs/windows
>
>
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > Tom Lane wrote:
> >> What about the question of whether $(DESTDIR) belongs there or not?
> >> I think we had concluded that PGXS shouldn't ever use $(DESTDIR),
> >> because that's only for install-time stuff.
>
> > I considered that a separate issue and didn't explore it,
> but I think
> > you are right that $(DESTDIR) makes no sense so I will remove it.
>
> > Patch attached.
>
> If these are bogus then so are the uses in Makefile.aix and
> Makefile.darwin.
>
> I'm a bit bothered by the ones in pgxs.mk, too, although I suspect we
> have to leave those there for the benefit of contrib?
>
>             regards, tom lane
>
> ---------------------------(end of
> broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings
>

Re: [HACKERS] pgxs/windows

From
Tom Lane
Date:
"Rocco Altier" <RoccoA@Routescape.com> writes:
> If we take the DESTDIR out of here, where should we get the POSTGRES_IMP
> file from?

In the PGXS case, $(bindir) has been gotten from pg_config, and it's
correct as-is by definition.  Adding $(DESTDIR) to it cannot be right.

            regards, tom lane