Re: ecpg build now breaks mingw - Mailing list pgsql-hackers

From Tom Lane
Subject Re: ecpg build now breaks mingw
Date
Msg-id 21915.1191534048@sss.pgh.pa.us
Whole thread Raw
In response to ecpg build now breaks mingw  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-hackers
Andrew Dunstan <andrew@dunslane.net> writes:
> dllwrap  -o libecpg.dll --dllname libecpg.dll  --def libecpgdll.def execute.o typename.o descriptor.o data.o error.o
prepare.omemory.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


pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: First steps with 8.3 and autovacuum launcher
Next
From: Andrew Dunstan
Date:
Subject: Re: 8.4 TODO item: make src/port support libpq and ecpg directly