<pre style='margin:0'>
Marcus Calhoun-Lopez (MarcusCalhoun-Lopez) pushed a commit to branch master
in repository macports-ports.
</pre>
<p><a href="https://github.com/macports/macports-ports/commit/2b2840e6a55e8be3c6e97790d9b3eef0eaef1277">https://github.com/macports/macports-ports/commit/2b2840e6a55e8be3c6e97790d9b3eef0eaef1277</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'> new 2b2840e qt5,qmake: fix account for CLT SDK and set use_xcode
</span>2b2840e is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 2b2840e6a55e8be3c6e97790d9b3eef0eaef1277
</span>Author: satraul <satraul@gmail.com>
AuthorDate: Tue Aug 6 21:14:43 2019 +0700
<span style='display:block; white-space:pre;color:#404040;'> qt5,qmake: fix account for CLT SDK and set use_xcode
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> References https://trac.macports.org/ticket/58779
</span>---
_resources/port1.0/group/qmake5-1.0.tcl | 14 ++++++++++++++
aqua/qt5/Portfile | 8 +++++++-
2 files changed, 21 insertions(+), 1 deletion(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/qmake5-1.0.tcl b/_resources/port1.0/group/qmake5-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 38c4c34..0e30697 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/qmake5-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/qmake5-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -29,6 +29,15 @@ configure.cmd ${qt_qmake_cmd}
</span> configure.pre_args-replace --prefix=${prefix} "PREFIX=${prefix}"
configure.universal_args-delete --disable-dependency-tracking
<span style='display:block; white-space:pre;background:#e0ffe0;'>+platform macosx {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # Use Xcode on macOS <= 10.9 (os.major 13) because CLT doesn't ship with an SDK on 10.9-
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # Better way is to just check if CLT SDK works correctly rather than hardcode OS
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # See: https://trac.macports.org/ticket/58779
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if { [info exists use_xcode] && ${os.major} <= 13 } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ use_xcode yes
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+}
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span> pre-configure {
#
# -spec specifies build configuration (compiler, 32-bit/64-bit, etc.)
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -54,6 +63,11 @@ pre-configure {
</span> if { ![file exists ${sdks_dir}/MacOSX${configure.sdk_version}.sdk] } {
configure.sdk_version
}
<span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # same check as before, but if macports wants to use CLT's developer_dir instead of Xcode, then we check if the build OS version is available on CLT.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if { [info exists configure.developer_dir] && ${developer_dir} ne ${configure.developer_dir} && ![file exists ${configure.developer_dir}/SDKs/MacOSX${configure.sdk_version}.sdk] } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ configure.sdk_version
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span> }
# set QT and QMAKE values in a cache file
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/aqua/qt5/Portfile b/aqua/qt5/Portfile
</span><span style='display:block; white-space:pre;color:#808080;'>index f9a2d89..6a0ea8d 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/aqua/qt5/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/aqua/qt5/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -983,7 +983,13 @@ foreach {module module_info} [array get modules] {
</span> # starting with Xcode 7.0, the SDK for build OS version might not be available
# see https://trac.macports.org/ticket/53597
<span style='display:block; white-space:pre;background:#ffe0e0;'>- if { ![file exists ${configure.sdkroot}/MacOSX${configure.sdk_version}.sdk] } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set sdks_dir ${developer_dir}/Platforms/MacOSX.platform/Developer/SDKs
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if { ![file exists ${sdks_dir}/MacOSX${configure.sdk_version}.sdk] } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ configure.sdk_version
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # same check as before, but if macports wants to use CLT's developer_dir instead of Xcode, then we check if the build OS version is available on CLT.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if {[info exists configure.developer_dir] && ${developer_dir} ne ${configure.developer_dir} && ![file exists ${configure.developer_dir}/SDKs/MacOSX${configure.sdk_version}.sdk]} {
</span> configure.sdk_version
}
}
</pre><pre style='margin:0'>
</pre>