<pre style='margin:0'>
Ryan Schmidt (ryandesign) pushed a commit to branch master
in repository macports-ports.
</pre>
<p><a href="https://github.com/macports/macports-ports/commit/1600de47960c7d969c2c3f27954a16b792d83def">https://github.com/macports/macports-ports/commit/1600de47960c7d969c2c3f27954a16b792d83def</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 1600de47960 root5: Check os.platform before checking os.major
</span>1600de47960 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 1600de47960c7d969c2c3f27954a16b792d83def
</span>Author: Ryan Schmidt <ryandesign@macports.org>
AuthorDate: Mon May 10 11:18:06 2021 -0500
<span style='display:block; white-space:pre;color:#404040;'> root5: Check os.platform before checking os.major
</span>---
science/root5/Portfile | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/science/root5/Portfile b/science/root5/Portfile
</span><span style='display:block; white-space:pre;color:#808080;'>index 38b4af1b8c2..cbe871910b0 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/science/root5/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/science/root5/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -133,8 +133,8 @@ configure.args --etcdir=${install_prefix}/etc/root \
</span> --enable-builtin-freetype
# Does not build using MacOS10.14 SDK. So use 10.13 instead
<span style='display:block; white-space:pre;background:#ffe0e0;'>-if { ${os.major} >= 18 } {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- configure.args-append --with-macosxvers=10.13
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+if {${os.platform} eq "darwin" && ${os.major} >= 18} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ configure.args-append --with-macosxvers=10.13
</span> }
platform darwin {
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -203,25 +203,25 @@ universal_variant no
</span>
default_variants +ssl +xml +gsl +minuit2 +tmva +roofit +graphviz +opengl +soversion +http
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# Since ROOT 5.34.15 the cocoa backend is default on OSX 10.8 or newer
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Since ROOT 5.34.15 the cocoa backend is default on OS X 10.8 or newer
</span> # We also make it default on 10.7
<span style='display:block; white-space:pre;background:#ffe0e0;'>-if {${os.major} > 10} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+if {${os.platform} eq "darwin" && ${os.major} > 10} {
</span> set default_gui_variant +cocoa
} else {
set default_gui_variant +x11
}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# ROOT5 is no longer support from macOS 10.15
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-if { ${os.major} > 18 } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# ROOT5 is no longer supported from macOS 10.15
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+if {${os.platform} eq "darwin" && ${os.major} > 18} {
</span> known_fail yes
pre-fetch {
<span style='display:block; white-space:pre;background:#ffe0e0;'>- ui_error "${name} is not supported on macOS 10.15 onwards. Please migrate to ROOT6."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- return -code error "Unsupported OSX version"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ ui_error "${name} is not supported on macOS 10.15 or later. Please migrate to ROOT6."
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ return -code error "Unsupported macOS version"
</span> }
}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# macOS10.14 Requires optional SDK header package to be installed
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-if { ${os.major} == 18 && ![file exists /usr/include] } {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# macOS 10.14 requires optional SDK header package to be installed
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+if {${os.platform} eq "darwin" && ${os.major} == 18 && ![file exists /usr/include] } {
</span> pre-fetch {
ui_error "${name} requires the optional Xcode SDK header package to be installed."
ui_error "See https://apple.stackexchange.com/questions/337940/why-is-usr-include-missing-i-have-xcode-and-command-line-tools-installed-moja for more details."
</pre><pre style='margin:0'>
</pre>