diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2019-10-21 12:32:35 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2019-10-21 12:32:35 -0400 |
| commit | 44273ce4f664a1bb34ef8ea9359237344cd6aaec (patch) | |
| tree | cd524c164b5b043cb21207992ed454aaacade4de /src/makefiles/Makefile.cygwin | |
| parent | 80831bcdbe80a6ca7f22105e32c2cbb54e125c4c (diff) | |
| download | postgresql-44273ce4f664a1bb34ef8ea9359237344cd6aaec.tar.gz | |
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/makefiles/Makefile.cygwin')
| -rw-r--r-- | src/makefiles/Makefile.cygwin | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/makefiles/Makefile.cygwin b/src/makefiles/Makefile.cygwin index f274d802b1..81089d6257 100644 --- a/src/makefiles/Makefile.cygwin +++ b/src/makefiles/Makefile.cygwin @@ -12,7 +12,6 @@ LIBS:=$(filter-out -lm -lc, $(LIBS)) AROPT = crs DLSUFFIX = .dll -CFLAGS_SL = override CPPFLAGS += -DWIN32_STACK_RLIMIT=$(WIN32_STACK_RLIMIT) |