<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/424d5f48bcf25ae10ef9a0ae51e5c4b3b8cf47ed">https://github.com/macports/macports-ports/commit/424d5f48bcf25ae10ef9a0ae51e5c4b3b8cf47ed</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 424d5f48bcf25ae10ef9a0ae51e5c4b3b8cf47ed
</span>Author: Marcus Calhoun-Lopez <mcalhoun@macports.org>
AuthorDate: Sat Aug 11 22:03:45 2018 -0700

<span style='display:block; white-space:pre;color:#404040;'>    qt5-qtwebengine: allow Xcode versions >= 10.0
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    See https://codereview.qt-project.org/#/c/234617/
</span><span style='display:block; white-space:pre;color:#404040;'>    See https://bugreports.qt.io/browse/QTBUG-69476
</span><span style='display:block; white-space:pre;color:#404040;'>    Fixes https://trac.macports.org/ticket/56829
</span>---
 aqua/qt5/Portfile                             |  3 ++
 aqua/qt5/files/patch-qtwebengine_xcode10.diff | 66 +++++++++++++++++++++++++++
 2 files changed, 69 insertions(+)

<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 711ae2b..09835cb 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;'>@@ -1440,6 +1440,9 @@ foreach {module module_info} [array get modules] {
</span>                 build.env-append      CC=${configure.cc}
                 configure.args-append QMAKE_LINK=${configure.cxx}
 
<span style='display:block; white-space:pre;background:#e0ffe0;'>+                # see https://trac.macports.org/ticket/56829
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                patchfiles-append patch-qtwebengine_xcode10.diff
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span>                 # the following libraries, even when available, do not seem to affect the build
                 # so turn them off
                 configure.post_args-append  \
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/aqua/qt5/files/patch-qtwebengine_xcode10.diff b/aqua/qt5/files/patch-qtwebengine_xcode10.diff
</span>new file mode 100644
<span style='display:block; white-space:pre;color:#808080;'>index 0000000..fc5040d
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>--- /dev/null
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/aqua/qt5/files/patch-qtwebengine_xcode10.diff
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -0,0 +1,66 @@
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+From 0fc07d2943753f444f3eeccd9fb1dfde0938cb70 Mon Sep 17 00:00:00 2001
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+From: Alexandru Croitor <alexandru.croitor@qt.io>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Date: Mon, 16 Jul 2018 13:24:51 +0200
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Subject: [PATCH 1/1] Fix Xcode version check to work with major versions >= 10
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+MIME-Version: 1.0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Content-Type: text/plain; charset=utf8
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Content-Transfer-Encoding: 8bit
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+We used lessThan for the Xcode version check, which started to fail
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+when comparing Xcode 10 with Xcode 7.3, because lessThan first tries
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+to convert the arguments to ints and if that fails, it does string
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+comparison instead.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Rewrite the code to be similar to the SDK checks.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+We can't use the qmake versionAtLeast function because it was added
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+in Qt 5.10, and we still need to be able to build against Qt 5.9.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Task-number: QTBUG-69476
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Change-Id: I831a683ee676838a4d531a4d6e715182e9e4193d
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+Reviewed-by: Michael Brüning <michael.bruning@qt.io>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+---
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ mkspecs/features/platform.prf | 22 +++++++++++++++++++++-
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ 1 file changed, 21 insertions(+), 1 deletion(-)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+diff --git a/mkspecs/features/platform.prf b/mkspecs/features/platform.prf
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+index 614f7fc92..c790c9fa6 100644
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+--- mkspecs/features/platform.prf
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++++ mkspecs/features/platform.prf
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+@@ -38,7 +38,7 @@ defineTest(isPlatformSupported) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+          return(false)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+     }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+   } else:osx {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+-    lessThan(QMAKE_XCODE_VERSION, 7.3) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    !isMinXcodeVersion(7, 3) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+       skipBuild("Using Xcode version $$QMAKE_XCODE_VERSION, but at least version 7.3 is required to build Qt WebEngine.")
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+       return(false)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+     }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+@@ -128,6 +128,26 @@ defineTest(isMinOSXSDKVersion) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+     return(false)
</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;'>++defineTest(isMinXcodeVersion) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    requested_major = $$1
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    requested_minor = $$2
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    requested_patch = $$3
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    isEmpty(requested_minor): requested_minor = 0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    isEmpty(requested_patch): requested_patch = 0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    target_var = QMAKE_XCODE_VERSION
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    major_version = $$section($$target_var, ., 0, 0)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    minor_version = $$section($$target_var, ., 1, 1)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    patch_version = $$section($$target_var, ., 2, 2)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    isEmpty(minor_version): minor_version = 0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    isEmpty(patch_version): patch_version = 0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    greaterThan(major_version, $$requested_major):return(true)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    equals(major_version, $$requested_major):greaterThan(minor_version, $$requested_minor):return(true)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    equals(major_version, $$requested_major):equals(minor_version, $$requested_minor):!lessThan(patch_version, $$requested_patch):return(true)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++    return(false)
</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;'>+ defineTest(isMinWinSDKVersion) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+     requested_major = $$1
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+     requested_minor = $$2
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+-- 
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+2.16.3
</span></pre><pre style='margin:0'>

</pre>