From 30657b796c7fdcaf9c0eb9ac53d4bab6399eb65b Mon Sep 17 00:00:00 2001 From: Tom Lane <tgl@sss.pgh.pa.us> Date: Wed, 12 Feb 2014 11:53:07 -0500 Subject: [PATCH] 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. --- src/template/cygwin | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/template/cygwin b/src/template/cygwin index 3948fab6410..8441bf9691f 100644 --- 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" -- GitLab