]> granicus.if.org Git - postgresql/commit
Select CFLAGS_SL at configure time, not in platform-specific Makefiles.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 21 Oct 2019 16:32:35 +0000 (12:32 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 21 Oct 2019 16:32:35 +0000 (12:32 -0400)
commit44273ce4f664a1bb34ef8ea9359237344cd6aaec
treecd524c164b5b043cb21207992ed454aaacade4de
parent80831bcdbe80a6ca7f22105e32c2cbb54e125c4c
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
21 files changed:
configure
configure.in
src/Makefile.global.in
src/makefiles/Makefile.cygwin
src/makefiles/Makefile.freebsd
src/makefiles/Makefile.hpux
src/makefiles/Makefile.linux
src/makefiles/Makefile.netbsd
src/makefiles/Makefile.openbsd
src/makefiles/Makefile.solaris
src/makefiles/Makefile.win32
src/template/aix
src/template/cygwin
src/template/darwin
src/template/freebsd
src/template/hpux
src/template/linux
src/template/netbsd
src/template/openbsd
src/template/solaris
src/template/win32