]> granicus.if.org Git - postgresql/commit
Prevent accidental linking of system-supplied copies of libpq.so etc.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 9 Jul 2018 21:23:31 +0000 (17:23 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 9 Jul 2018 21:23:31 +0000 (17:23 -0400)
commitccc286da1d35b5d57a1e3322027905c00507d38f
tree5d17712252780e0be4d0bb1b021ff4e3fd8444de
parent619dea4678c345d119036311c503b55b974695d3
Prevent accidental linking of system-supplied copies of libpq.so etc.

Back-patch commit dddfc4cb2, which broke LDFLAGS and related Makefile
variables into two parts, one for within-build-tree library references and
one for external libraries, to ensure that the order of -L flags has all
of the former before all of the latter.  This turns out to fix a problem
recently noted on buildfarm member peripatus, that we attempted to
incorporate code from libpgport.a into a shared library.  That will fail on
platforms that are sticky about putting non-PIC code into shared libraries.
(It's quite surprising we hadn't seen such failures before, since the code
in question has been like that for a long time.)

I think that peripatus' problem could have been fixed with just a subset
of this patch; but since the previous issue of accidentally linking to the
wrong copy of a Postgres shlib seems likely to bite people in the field,
let's just back-patch the whole change.  Now that commit dddfc4cb2 has
survived some beta testing, I'm less afraid to back-patch it than I was
at the time.

This also fixes undesired inclusion of "-DFRONTEND" in pg_config's CPPFLAGS
output (in 9.6 and up) and undesired inclusion of "-L../../src/common" in
its LDFLAGS output (in all supported branches).

Back-patch to v10 and older branches; this is already in v11.

Discussion: https://postgr.es/m/20180704234304.bq2dxispefl65odz@ler-imac.local
29 files changed:
contrib/dblink/Makefile
contrib/hstore_plperl/Makefile
contrib/hstore_plpython/Makefile
contrib/ltree_plpython/Makefile
contrib/oid2name/Makefile
contrib/postgres_fdw/Makefile
contrib/spi/Makefile
contrib/vacuumlo/Makefile
src/Makefile.global.in
src/Makefile.shlib
src/backend/replication/libpqwalreceiver/Makefile
src/bin/pg_basebackup/Makefile
src/bin/pg_ctl/Makefile
src/bin/pg_dump/Makefile
src/bin/pg_rewind/Makefile
src/bin/pg_upgrade/Makefile
src/bin/pgbench/Makefile
src/bin/psql/Makefile
src/bin/scripts/Makefile
src/common/Makefile
src/interfaces/ecpg/compatlib/Makefile
src/interfaces/ecpg/ecpglib/Makefile
src/interfaces/ecpg/pgtypeslib/Makefile
src/interfaces/ecpg/test/Makefile.regress
src/interfaces/ecpg/test/compat_informix/Makefile
src/interfaces/libpq/test/Makefile
src/makefiles/pgxs.mk
src/test/examples/Makefile
src/tools/findoidjoins/Makefile