Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit68fc227

Browse files
committed
Back off using -isysroot on Darwin.
Rethink the solution applied in commit5e22171 to get PL/Tcl tobuild on macOS Mojave. I feared that adding -isysroot globally mighthave undesirable consequences, and sure enough Jakob Egger reportedone: it complicates building extensions with a different Xcode versionthan was used for the core server. (I find that a risky propositionin general, but apparently it works most of the time, so we shouldn'tbreak it if we don't have to.)We'd already adopted the solution for PL/Perl of inserting the sysrootpath directly into the -I switches used to find Perl's headers, and wecan do the same thing for PL/Tcl by changing the -iwithsysroot switchthat Apple's tclConfig.sh reports. This restricts the risks to PL/Perland PL/Tcl themselves and directly-dependent extensions, which is a lotmore pleasing in general than a global -isysroot switch.Along the way, tighten the test to see if we need to inject the sysrootpath into $perl_includedir, as I'd speculated about upthread but notgotten round to doing.As before, back-patch to all supported versions.Discussion:https://postgr.es/m/20840.1537850987@sss.pgh.pa.us
1 parent93ca02e commit68fc227

File tree

3 files changed

+17
-6
lines changed

3 files changed

+17
-6
lines changed

‎configure

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9778,7 +9778,7 @@ PL/Perl." "$LINENO" 5
97789778
perl_includedir="$perl_archlibexp"
97799779
# ... but on some macOS versions, we must look under $PG_SYSROOT instead
97809780
if test x"$PG_SYSROOT" != x"" ; then
9781-
if test -d "$PG_SYSROOT$perl_archlibexp" ; then
9781+
if test -f "$PG_SYSROOT$perl_archlibexp/CORE/perl.h" ; then
97829782
perl_includedir="$PG_SYSROOT$perl_archlibexp"
97839783
fi
97849784
fi
@@ -18114,6 +18114,11 @@ eval TCL_SHARED_BUILD=\"$TCL_SHARED_BUILD\"
1811418114
as_fn_error $? "cannot build PL/Tcl because Tcl is not a shared library
1811518115
Use --without-tcl to disable building PL/Tcl." "$LINENO" 5
1811618116
fi
18117+
# Some macOS versions report an include spec that uses -iwithsysroot.
18118+
# We don't really want to use -isysroot, so translate that if we can.
18119+
if test x"$PG_SYSROOT" != x"" ; then
18120+
TCL_INCLUDE_SPEC="`echo "$TCL_INCLUDE_SPEC" | sed "s|-iwithsysroot */|-I $PG_SYSROOT/|"`"
18121+
fi
1811718122
# now that we have TCL_INCLUDE_SPEC, we can check for <tcl.h>
1811818123
ac_save_CPPFLAGS=$CPPFLAGS
1811918124
CPPFLAGS="$TCL_INCLUDE_SPEC $CPPFLAGS"

‎configure.in

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1048,7 +1048,7 @@ PL/Perl.])
10481048
perl_includedir="$perl_archlibexp"
10491049
# ... but on some macOS versions, we must look under $PG_SYSROOT instead
10501050
if test x"$PG_SYSROOT" != x"" ; then
1051-
if test -d "$PG_SYSROOT$perl_archlibexp" ; then
1051+
if test -f "$PG_SYSROOT$perl_archlibexp/CORE/perl.h" ; then
10521052
perl_includedir="$PG_SYSROOT$perl_archlibexp"
10531053
fi
10541054
fi
@@ -2211,6 +2211,11 @@ if test "$with_tcl" = yes; then
22112211
AC_MSG_ERROR([cannot build PL/Tcl because Tcl is not a shared library
22122212
Use --without-tcl to disable building PL/Tcl.])
22132213
fi
2214+
# Some macOS versions report an include spec that uses -iwithsysroot.
2215+
# We don't really want to use -isysroot, so translate that if we can.
2216+
if test x"$PG_SYSROOT" != x"" ; then
2217+
TCL_INCLUDE_SPEC="`echo "$TCL_INCLUDE_SPEC" | sed "s|-iwithsysroot */|-I $PG_SYSROOT/|"`"
2218+
fi
22142219
# now that we have TCL_INCLUDE_SPEC, we can check for <tcl.h>
22152220
ac_save_CPPFLAGS=$CPPFLAGS
22162221
CPPFLAGS="$TCL_INCLUDE_SPEC $CPPFLAGS"

‎src/template/darwin

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,15 @@
33
# Note: Darwin is the original code name for macOS, also known as OS X.
44
# We still use "darwin" as the port name, partly because config.guess does.
55

6-
# Select where system include files should be sought.
6+
# Select where some include files should be sought.
7+
# We may eventually be forced to use "-isysroot" with this value,
8+
# but for now, it only affects Perl and Tcl include files.
79
if test x"$PG_SYSROOT" = x"" ; then
810
PG_SYSROOT=`xcodebuild -version -sdk macosx Path 2>/dev/null`
911
fi
12+
# Old xcodebuild versions may produce garbage, so validate the result.
1013
if test x"$PG_SYSROOT" != x"" ; then
11-
if test -d "$PG_SYSROOT" ; then
12-
CPPFLAGS="$CPPFLAGS -isysroot $PG_SYSROOT"
13-
else
14+
if test \! -d "$PG_SYSROOT" ; then
1415
PG_SYSROOT=""
1516
fi
1617
fi

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp