Re: Cygwin plpython patch

From: Jason Tishler <Jason(dot)Tishler(at)dothill(dot)com>
To: Ernesto Revilla <aerd(at)retemail(dot)es>
Cc: pgsql-cygwin(at)postgresql(dot)org
Subject: Re: Cygwin plpython patch
Date: 2001-05-16 01:17:26
Message-ID: 20010515211726.A1071@dothill.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-cygwin pgsql-patches

Erny,

On Wed, May 16, 2001 at 01:27:15AM +0200, Ernesto Revilla wrote:
> 1. I downloaded all plpython files from CVS, including Makefile
> 2. " and installed the global Makefile (Makefile.global) and the
> configure script.
> 3. " and installed the Win.h of the src/backend/port/dynloader and
> other files
> this all is under 7.1.1 CVS version
>
> When I do a make, plpython does not compile, because:
> 1. plpython is not automatically included in the make process

The automatic building of plpython is not enabled yet. From
src/pl/Makefile, we have:

# Doesn't build portably yet.
#ifeq ($(with_python), yes)
#DIRS += plpython
#endif

Hence, you will have to cd to src/pl/plpython and run make manually.

> 2. for strange reasons -lpostgres is not included in the dllwrap command
> line. (I did a new ./configure --with-python and make clean before
> beginning).
> 3. dllwrap (or dlltool) complains that there are references to unknown
> symbols: _Py_NoneStruct, _Py_TrueStruct, PyString_Type, PyType_type, and 2
> others. This is because it uses /usr/lib/python2.1/config/libpython2.1.dll.a
> which seems not to define these symbols. So I change this and link it
> against /bin/libpython.2.1.dll (this is where it is installed by default).
> Now the linker runs fine, without any error or warning.
>
> 4. When I create the language (absolute reference to /lib/plpython.dll) and
> I run a testproc something like 'CREATE FUNCION kk() RETURNS int4 AS 'return
> 1' LANGUAGE 'plpython'; the server process crashes.
>
> Any ideas?

Did you forget to apply my patch? I have only submitted it to
pgsql-patches, it has not been accepted into CVS. Without my patch you
would get errors like the above. With my patch, plpython should build
OOTB -- you should not have to do unrecommended things like #3 above.

Jason

--
Jason Tishler
Director, Software Engineering Phone: +1 (732) 264-8770 x235
Dot Hill Systems Corp. Fax: +1 (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

In response to

Browse pgsql-cygwin by date

  From Date Subject
Next Message Jason Tishler 2001-05-17 20:44:39 Re: PostgreSQL with Cygwin
Previous Message Jason Tishler 2001-05-15 18:28:21 Re: postgresql client under win 2k

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2001-05-16 02:35:46 Re: [PATCHES] DatabaseMetaData.getIndexInfo() added
Previous Message Bruce Momjian 2001-05-16 01:08:42 Re: Fix for SQLStatistics