<pre style='margin:0'>
Jeremy Huddleston Sequoia (jeremyhu) pushed a commit to branch master
in repository macports-ports.
</pre>
<p><a href="https://github.com/macports/macports-ports/commit/afefdd0101048637d3b915c04e840fd0812ab150">https://github.com/macports/macports-ports/commit/afefdd0101048637d3b915c04e840fd0812ab150</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 afefdd0 muniversal: Use cctools-provided nm if installed
</span>afefdd0 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit afefdd0101048637d3b915c04e840fd0812ab150
</span>Author: Jeremy Huddleston Sequoia <jeremyhu@macports.org>
AuthorDate: Sun Dec 25 20:53:09 2016 -0800
<span style='display:block; white-space:pre;color:#404040;'> muniversal: Use cctools-provided nm if installed
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> I have my doubts about whether NM should be set at all, but until that's
</span><span style='display:block; white-space:pre;color:#404040;'> worked out, update the hack to pick up nm from cctools rather than
</span><span style='display:block; white-space:pre;color:#404040;'> /usr/bin if it's installed.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> This addresses build failures with the +universal variant when using
</span><span style='display:block; white-space:pre;color:#404040;'> macports-clang-3.9 on older systems with muniversal ports (eg: gnutls).
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@macports.org>
</span>---
_resources/port1.0/group/muniversal-1.0.tcl | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/muniversal-1.0.tcl b/_resources/port1.0/group/muniversal-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index b523177..5c37802 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/muniversal-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/muniversal-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -198,7 +198,13 @@ variant universal {
</span>
configure {
# Fix inability to find nm when cross-compiling (#22224, #23431, #23687, #24477, et al)
<span style='display:block; white-space:pre;background:#ffe0e0;'>- configure.env-append NM=/usr/bin/nm
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # TODO: I suspect we should remove this. base doesn't do this, so I don't see why muniversal should.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # This also seems like a but in certain versions of autoconf, so ports should just autoreconf (or patch).
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if {[file exists ${prefix}/bin/nm]} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ configure.env-append NM=${prefix}/bin/nm
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ } else {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ configure.env-append NM=/usr/bin/nm
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ }
</span>
foreach arch ${universal_archs_to_use} {
ui_info "$UI_PREFIX [format [msgcat::mc "Configuring %1\$s for architecture %2\$s"] ${subport} ${arch}]"
</pre><pre style='margin:0'>
</pre>