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 |
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 ...) |