[151078] contrib/mp-buildbot/mpbb-install-dependencies

[cal at macports.org]

Revision: 151078
          https://trac.macports.org/changeset/151078
Author:   cal at macports.org
Date:     2016-08-06 09:12:29 -0700 (Sat, 06 Aug 2016)
Log Message:
-----------
mp-buildbot: Improve install-dependencies output

Fix off-by-one error in substring extraction of the variant spec and add
a unique prefix to improve output readability. This turns
 | Installing dependency 'expat', variants: ' +universal'
into
 | ----> Installing dependency 'expat', variants: '+universal'
which is easier to grep and does not contain useless whitespace.

Modified Paths:
--------------
    contrib/mp-buildbot/mpbb-install-dependencies

Modified: contrib/mp-buildbot/mpbb-install-dependencies
===================================================================
--- contrib/mp-buildbot/mpbb-install-dependencies	2016-08-06 16:11:42 UTC (rev 151077)
+++ contrib/mp-buildbot/mpbb-install-dependencies	2016-08-06 16:12:29 UTC (rev 151078)
@@ -37,9 +37,9 @@
         # Split portname +variant1+variant2 into portname and variants, where
         # the variants are optional.
         depname=${dependency%% *}
-        depvariants=${dependency:${#depname}}
+        depvariants=${dependency:${#depname}+1}
 
-        echo "Installing dependency '${depname}', variants: '${depvariants}'"
+        echo "----> Installing dependency '${depname}', variants: '${depvariants}'"
         if ! "${option_prefix}/bin/port" -d install --unrequested "$depname" $depvariants; then
             echo "Build of dependency '${depname}' failed, cleaning up..." >&2
             cleanup
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20160806/29879cb7/attachment-0001.html>