Better messaging around auto tool install

getlantern · Feb 21, 2015 · dd355cf · dd355cf
1 parent 499081f
commit dd355cf
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/genassets.bash b/genassets.bash
@@ -7,10 +7,11 @@ APP="$LANTERN_UI/app"
 DIST="$LANTERN_UI/dist"
 
 if [ ! -d $DIST ] || [ $APP -nt $DIST ]; then
-    echo "Updating dist folder"
-
     # Install gulp (requires nodejs)
-    which gulp || npm install gulp -g
+    echo "Installing gulp tool if necessary (requires nodejs)"
+    which gulp || npm install -g gulp
+
+    echo "Updating dist folder"
     cd $LANTERN_UI
     npm install
     rm -Rf dist

diff --git a/package_osx.bash b/package_osx.bash
@@ -10,10 +10,10 @@ then
     die "$0: Version required"
 fi
 
-# Install svgexport tool (requires nodejs)
-which svgexport || npm install svgexport -g
+echo "Installing svgexport tool if necessary (requires nodejs)"
+which svgexport || npm install -g svgexport
 
-# Install appdmg tool (requires nodejs)
+echo "Installing appdmg tool if necessary (requires nodejs)"
 which appdmg || npm install -g appdmg
 
 binary="lantern_darwin_amd64"