From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: ecpg build now breaks mingw |
Date: | 2007-10-04 21:40:48 |
Message-ID: | 21915.1191534048@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> dllwrap -o libecpg.dll --dllname libecpg.dll --def libecpgdll.def execute.o typename.o descriptor.o data.o error.o prepare.o memory.o connect.o misc.o path.o strlcpy.o snprintf.o -L../pgtypeslib -L../../../../src/interfaces/libpq -L../../../../src/port -L/mingw/lib -lpgtypes -lpq -lm -lshfolder
> path.o(.text+0x741): In function `get_progname':
> C:/msys/1.0/local/pgbuildfarm/buildroot/HEAD/pgsql.836/src/interfaces/ecpg/ecpglib/path.c:413: undefined reference to `pg_strcasecmp'
I just fixed that.
> I presume that this symbol was previously exported by the pgtypes lib but now we have an explicit exports list is no longer.
> Is it just a matter of adding a line to the exports.txt file?
Well, that would be the solution if we intended to make pg_strcasecmp
part of libpgtype's official API, but that doesn't seem like a good plan
at all. Instead, the right thing is for ecpglib to pull its own copy
from src/port/.
BTW, it looks like this dependency exists only on Cygwin/Win32, not
other platforms, which is no doubt why we didn't see it before.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2007-10-04 22:16:13 | Re: 8.4 TODO item: make src/port support libpq and ecpg directly |
Previous Message | Tom Lane | 2007-10-04 21:33:43 | 8.4 TODO item: make src/port support libpq and ecpg directly |