diff options
| author | Tom Lane | 2019-10-21 16:32:35 +0000 |
|---|---|---|
| committer | Tom Lane | 2019-10-21 16:32:35 +0000 |
| commit | 44273ce4f664a1bb34ef8ea9359237344cd6aaec (patch) | |
| tree | cd524c164b5b043cb21207992ed454aaacade4de /src/template/cygwin | |
| parent | 80831bcdbe80a6ca7f22105e32c2cbb54e125c4c (diff) | |
Select CFLAGS_SL at configure time, not in platform-specific Makefiles.
Move the platform-dependent logic that sets CFLAGS_SL from
src/makefiles/Makefile.foo to src/template/foo, so that the value
is determined at configure time and thus is available while running
configure's tests.
On a couple of platforms this might save a few microseconds of build
time by eliminating a test that make otherwise has to do over and over.
Otherwise it's pretty much a wash for build purposes; in particular,
this makes no difference to anyone who might be overriding CFLAGS_SL
via a make option.
This patch in itself does nothing with the value and thus should not
change any behavior, though you'll probably have to re-run configure
to get a correctly updated Makefile.global. We'll use the new
configure variable in a follow-on patch.
Per gripe from Kyotaro Horiguchi. Back-patch to all supported branches,
because the follow-on patch is a portability bug fix.
Discussion: https://postgr.es/m/20191010.144533.263180400.horikyota.ntt@gmail.com
Diffstat (limited to 'src/template/cygwin')
| -rw-r--r-- | src/template/cygwin | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/template/cygwin b/src/template/cygwin index b6ea0ded522..79a108145ff 100644 --- a/src/template/cygwin +++ b/src/template/cygwin @@ -2,6 +2,9 @@ SRCH_LIB="/usr/local/lib" +# Extra CFLAGS for code that will go into a shared library +CFLAGS_SL="" + # --allow-multiple-definition is required to link pg_dump because it finds # pg_toupper() etc. in both libpq and pgport # we'd prefer to use --disable-auto-import to match MSVC linking behavior, |