Re: plperl win32 - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: plperl win32
Date
Msg-id 200407161918.i6GJIjH12855@candle.pha.pa.us
Whole thread Raw
In response to plperl win32  ("Magnus Hagander" <mha@sollentuna.net>)
List pgsql-patches

Attached is the modified patch I applied.  Thanks.

---------------------------------------------------------------------------


Magnus Hagander wrote:
> Here is a patch required to build plperl with win32. The issues were:
>
> * perl_useshrplib gets set to "yes" and not to "true". I assume it's set
> to "true" on unix, so I left both.
> * Need to translate backslashes into slashes
> * The linker config coming out of perl was for MSVC and not for mingw
>
>
> Some of this is pretty ugly stuff - the reassigning into a second
> variable etc. If somebody with a little better "makefile knowledge"
> would clean that up (if it can be, but it should be possible), please
> do!
>
>
> //Magnus
>
>  <<plperl_win32.patch>>

Content-Description: plperl_win32.patch

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 8: explain analyze is your friend

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
Index: src/pl/plperl/GNUmakefile
===================================================================
RCS file: /cvsroot/pgsql-server/src/pl/plperl/GNUmakefile,v
retrieving revision 1.14
diff -c -c -r1.14 GNUmakefile
*** src/pl/plperl/GNUmakefile    5 Jul 2004 23:24:12 -0000    1.14
--- src/pl/plperl/GNUmakefile    16 Jul 2004 19:16:57 -0000
***************
*** 8,13 ****
--- 8,16 ----
  ifeq ($(perl_useshrplib),true)
  shared_libperl = yes
  endif
+ ifeq ($(perl_useshrplib),yes)
+ shared_libperl = yes
+ endif

  # If we don't have a shared library and the platform doesn't allow it
  # to work without, we have to skip it.
***************
*** 18,24 ****
  override CFLAGS := $(filter-out -Wall -Wmissing-declarations -Wmissing-prototypes, $(CFLAGS))
  endif

! override CPPFLAGS := -I$(srcdir) -I$(perl_archlibexp)/CORE $(CPPFLAGS)


  NAME = plperl
--- 21,33 ----
  override CFLAGS := $(filter-out -Wall -Wmissing-declarations -Wmissing-prototypes, $(CFLAGS))
  endif

! ifeq ($(PORTNAME), win32)
! perl_archlibexp := $(subst \,/,$(perl_archlibexp))
! perl_privlibexp := $(subst \,/,$(perl_privlibexp))
! perl_embed_ldflags := -L $(perl_archlibexp)/CORE -lperl58
! endif
!
! override CPPFLAGS := -I$(srcdir) $(CPPFLAGS) -I$(perl_archlibexp)/CORE


  NAME = plperl

pgsql-patches by date:

Previous
From: Klaus Naumann
Date:
Subject: Re: Show tablespace name in pg_tables and pg_indexes
Next
From: "Matthew T. O'Connor"
Date:
Subject: Re: pg_autovacuum integration attempt #2