Re: Python (PyGreSQL) make install patch - Mailing list pgsql-patches

From Peter Eisentraut
Subject Re: Python (PyGreSQL) make install patch
Date
Msg-id Pine.LNX.4.30.0106222024430.727-200000@peter.localdomain
Whole thread Raw
In response to Re: Python (PyGreSQL) make install patch  (Jason Tishler <Jason.Tishler@dothill.com>)
Responses Re: Python (PyGreSQL) make install patch
List pgsql-patches
[The updated proposed new makefile for interfaces/python is attached.]

[May 30th] Jason Tishler writes:

> I like your proposed Makefile for many reasons: consistent with the
> rest of the PostgreSQL make infrastructure, easy to understand, DESTDIR
> support, etc.  However, PostgreSQL is now burdened with determining and
> maintaining the Python shared extension cruft for all supported platforms.
> I am quite willing to supply the Cygwin stuff, but you are on your own
> for other platforms like AIX, etc.

AFAICT, we build shared libraries on AIX okay.

> See attached for patches that are necessary for Cygwin.

> @@ -13,7 +13,7 @@ include $(top_srcdir)/src/Makefile.shlib
>  override CFLAGS += $(CFLAGS_SL)
>  # (Python also has a config.h file.  Be sure to use ours.)
>  override CPPFLAGS := -I$(libpq_srcdir) $(CPPFLAGS) -I$(python_includedir)
> -SHLIB_LINK = $(libpq)
> +SHLIB_LINK = $(libpq) $(python_libspec)
>
>  all: all-lib

I'm not sure about this change.  $(python_libspec) is to link an embedded
python, e.g., it includes -lpython itself.  I'm not sure this would work
everywhere, since the module will be loaded by a python interpreter, so
you could get conflicts.  Does this correspond to what the current "gmake
all" gives you?

>     $(exec-prefix)/lib/python$(version)/site-packages
>
> is the place for third-party modules.

I have since found supporting documentation for this.

--
Peter Eisentraut   peter_e@gmx.net   http://funkturm.homeip.net/~peter

Attachment

pgsql-patches by date:

Previous
From: Marko Kreen
Date:
Subject: Re: use GUC for cmdline
Next
From: Jason Tishler
Date:
Subject: Re: Python (PyGreSQL) make install patch