Remove --enable-auto-import linker switch in Cygwin build.

This is expected to make it start failing when contrib modules
reference non-PGDLLIMPORT'ed global variables, as the other Windows
build methods do.  Aside from the value of consistency, the underlying
implementation of this switch is pretty ugly and not really something
we want to rely on if we have to use PGDLLIMPORT anyway for MSVC.

diff --git a/src/template/cygwin b/src/template/cygwin
index 3948fab641011bed56753fe9f4d20a2ea8ee6b2e..8441bf9691f862d45b18447efd0449db90d636a7 100644 (file)
--- a/src/template/cygwin
+++ b/src/template/cygwin
@@ -3,6 +3,6 @@
 SRCH_LIB="/usr/local/lib"
 
 # --allow-multiple-definition is required to link pg_dump because it finds
-# pg_toupper() in both libpq and pgport
-# --enable-auto-import gets rid of a diagnostics linker message
-LDFLAGS="-Wl,--allow-multiple-definition -Wl,--enable-auto-import"
+# 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"