pgsql: Still further rethinking of build changes for macOS Mojave.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Still further rethinking of build changes for macOS Mojave.
Date: 2018-10-18 18:55:53
Message-ID: E1gDDSb-0001ZI-Bl@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Still further rethinking of build changes for macOS Mojave.

To avoid the sorts of problems complained of by Jakob Egger, it'd be
best if configure didn't emit any references to the sysroot path at all.
In the case of PL/Tcl, we can do that just by keeping our hands off the
TCL_INCLUDE_SPEC string altogether. In the case of PL/Perl, we need to
substitute -iwithsysroot for -I in the compile commands, which is easily
handled if we change to using a configure output variable that includes
the switch not only the directory name. Since PL/Tcl and PL/Python
already do it like that, this seems like good consistency cleanup anyway.

Hence, this replaces the advice given to Perl-related extensions in commit
5e2217131; instead of writing "-I$(perl_archlibexp)/CORE", they should
just write "$(perl_includespec)". (The old way continues to work, but not
on recent macOS.)

It's still the case that configure needs to be aware of the sysroot
path internally, but that's cleaner than what we had before.

As before, back-patch to all supported versions.

Discussion: https://postgr.es/m/20840.1537850987@sss.pgh.pa.us

Branch
------
REL9_6_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/1b92ca9e2c22fac54e4f77c80b6bbe8469ed472d

Modified Files
--------------
configure | 18 +++++++-----------
configure.in | 18 +++++++-----------
contrib/hstore_plperl/Makefile | 7 ++-----
src/Makefile.global.in | 2 +-
src/pl/plperl/GNUmakefile | 5 +++--
src/template/darwin | 5 ++---
6 files changed, 22 insertions(+), 33 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2018-10-19 01:01:13 pgsql: Refactor pid, random seed and start time initialization.
Previous Message Andrew Dunstan 2018-10-17 21:30:05 Re: pgsql: Add TAP tests for pg_verify_checksums