Re: PostgreSQL CVS does not compile under Cygwin (was Re: Cygwin test ...)

From: Jason Tishler <jason(at)tishler(dot)net>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pgsql-Cygwin <pgsql-cygwin(at)postgresql(dot)org>
Subject: Re: PostgreSQL CVS does not compile under Cygwin (was Re: Cygwin test ...)
Date: 2001-08-26 01:19:04
Message-ID: 20010825211904.J1556@dothill.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-cygwin

Bruce,

On Sat, Aug 25, 2001 at 08:15:45PM -0400, Bruce Momjian wrote:
> Can someone research this and figure out what the proper solution for
> this is? Seems we are going around in circles if we keep
> adding/removing DLLIMPORT.

I believe that the attached patch is the correct solution -- I apologize
for the gyrations. With the attached patch, Cygwin libpq++ builds
cleanly again. The root cause was that DLLIMPORT was defaulting to
__declspec(dllimport) since BUILDING_DLL was *not* defined when building
the libpq++ DLL.

Unfortunately, to test my patch requires changing the following makefile:

src/interfaces/libpq++/examples/Makefile

and the #includes in all of the *.cc to build against the source tree
instead of the following hardcoded installation directory structure:

/usr/local/pgsql

I was able to manually build

src/interfaces/libpq++/examples/testlibpq0.exe

against my Cygwin libpq++ without errors. However, I have not tried to
actually test testlibpq0.exe.

Is this sufficient? Or, do you want me to clean up libpq++/examples too?
(Or, is it silly to even ask? :,)) Let me know how you want to proceed and
I will submit a patch to pgsql-patches.

Thanks,
Jason

Attachment Content-Type Size
Makefile.win.diff text/plain 631 bytes

In response to

Responses

Browse pgsql-cygwin by date

  From Date Subject
Next Message Bruce Momjian 2001-08-26 03:09:58 Re: PostgreSQL CVS does not compile under Cygwin (was Re: Cygwin test ...)
Previous Message Bruce Momjian 2001-08-26 00:15:45 Re: PostgreSQL CVS does not compile under Cygwin (was Re: Cygwin test ...)