From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Select CFLAGS_SL at configure time, not in platform-specific Mak |
Date: | 2019-10-21 17:52:56 |
Message-ID: | E1iMbrU-0006zl-7e@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
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
Branch
------
REL_12_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/aa5bb828af5387e154a122b1b43ee873d92497a0
Modified Files
--------------
configure | 7 ++++---
configure.in | 9 ++++++---
src/Makefile.global.in | 1 +
src/makefiles/Makefile.cygwin | 1 -
src/makefiles/Makefile.freebsd | 2 --
src/makefiles/Makefile.hpux | 5 -----
src/makefiles/Makefile.linux | 2 --
src/makefiles/Makefile.netbsd | 2 --
src/makefiles/Makefile.openbsd | 2 --
src/makefiles/Makefile.solaris | 6 +-----
src/makefiles/Makefile.win32 | 1 -
src/template/aix | 5 +++++
src/template/cygwin | 3 +++
src/template/darwin | 3 +++
src/template/freebsd | 3 +++
src/template/hpux | 7 +++++++
src/template/linux | 3 +++
src/template/netbsd | 3 +++
src/template/openbsd | 4 ++++
src/template/solaris | 9 +++++++++
src/template/win32 | 3 +++
21 files changed, 55 insertions(+), 26 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2019-10-21 18:19:30 | pgsql: Deal with yet another issue related to "Norwegian (Bokmål)" loca |
Previous Message | Etsuro Fujita | 2019-10-21 08:40:01 | pgsql: Update obsolete comment. |