Re: Python (PyGreSQL) make install patch

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Jason Tishler <Jason(dot)Tishler(at)dothill(dot)com>, Pgsql-Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Python (PyGreSQL) make install patch
Date: 2001-05-25 15:19:14
Message-ID: Pine.LNX.4.30.0105251718230.758-100000@peter.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Bruce Momjian writes:

> Patch applied. Thanks. I know you are testing a newer one but it
> seemed worthwhile to get this in there.

No, no, no!

>
> > The attached patch changes src/interfaces/python/GNUmakefile to use the
> > value of DESTDIR like the rest (or at least most) of the PostgreSQL
> > makefiles. I found this problem when trying to package a pre-built
> > Cygwin PostgreSQL distribution, but this problem is platform independent.
> >
> > The problem manifests itself when one tries to install into a stagging
> > area (e.g., to build a tarball) instead of a real install. In this case,
> > pg.py and _pgmodule$(SO) still end up being installed in the configured
> > prefix directory ignoring the value of DESTDIR.
> >
> > Unfortunately, this patch does not handle the case where PostgreSQL
> > and Python are configured with different prefixes. Since the Python
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > Makefile is automatically generated and does not use DESTDIR, I believe
> > that this issue will be difficult to correct. If anyone has ideas on
> > how to fix this issue, then I'm quite willing to rework the patch to
> > take the suggestion into account.
> >
> > I tested this patch on Cygwin and Linux.
> >
> > Thanks,
> > Jason
> >
> > --
> > Jason Tishler
> > Director, Software Engineering Phone: 732.264.8770 x235
> > Dot Hill Systems Corp. Fax: 732.264.8798
> > 82 Bethany Road, Suite 7 Email: Jason(dot)Tishler(at)dothill(dot)com
> > Hazlet, NJ 07730 USA WWW: http://www.dothill.com
>
> [ Attachment, skipping... ]
>
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 3: if posting/reading through Usenet, please send an appropriate
> > subscribe-nomail command to majordomo(at)postgresql(dot)org so that your
> > message can get through to the mailing list cleanly
>
>

--
Peter Eisentraut peter_e(at)gmx(dot)net http://funkturm.homeip.net/~peter

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Peter Eisentraut 2001-05-25 15:31:20 Re: Python (PyGreSQL) make install patch
Previous Message Jason Tishler 2001-05-25 14:49:42 Re: Python (PyGreSQL) make install patch