From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Sun, 16 Feb 2014 20:14:04 +0000 (-0500)
Subject: Revert to using --enable-auto-import in Cygwin builds.
X-Git-Tag: REL9_4_BETA1~465
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8d6e2d4abf77c422714448e5f4270fdb1a84d973;p=postgresql

Revert to using --enable-auto-import in Cygwin builds.

Disabling auto-import requires that all libraries we use be careful about
declspecs for exported variables; and it seems they aren't.  This means
that Cygwin will not give us useful info about missing PGDLLIMPORT markers;
but it's probably sufficient that MSVC and Mingw builds do.
---

diff --git a/src/template/cygwin b/src/template/cygwin
index 8441bf9691..e484fe650c 100644
--- a/src/template/cygwin
+++ b/src/template/cygwin
@@ -4,5 +4,6 @@ SRCH_LIB="/usr/local/lib"
 
 # --allow-multiple-definition is required to link pg_dump because it finds
 # pg_toupper() etc. in both libpq and pgport
-# --disable-auto-import is to ensure we get MSVC-like linking behavior
-LDFLAGS="-Wl,--allow-multiple-definition -Wl,--disable-auto-import"
+# we'd prefer to use --disable-auto-import to match MSVC linking behavior,
+# but support for it in Cygwin is too haphazard
+LDFLAGS="-Wl,--allow-multiple-definition -Wl,--enable-auto-import"