Improving output of blt:alias. (#134)

acquia · Jun 9, 2016 · 41597dc · danepowell · Jun 13, 2016 · grasmash
1 parent ec641ba
commit 41597dc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/template/build/core/phing/tasks/blt.xml b/template/build/core/phing/tasks/blt.xml
@@ -10,6 +10,7 @@
   </target>
 
   <target name="blt:alias" description="Installs the BLT alias for command line usage.">
+    <!-- @todo Check to see if alias is already present. -->
     <echo>BLT can automatically create a Bash alias to make it easier to run BLT tasks.</echo>
     <echo>This alias may be created in .bash_profile or .bashrc depending on your system architecture.</echo>
     <propertyprompt propertyName="create_alias" defaultValue="y" useExistingValue="true" promptText="Create a bash alias now? (y/n)" />

diff --git a/template/scripts/blt/install-alias.sh b/template/scripts/blt/install-alias.sh
@@ -16,10 +16,13 @@ if [ ! -z "$DETECTED_PROFILE" ]; then
     exit
   fi
   DIR=$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )
-  cat $DIR/alias >> $DETECTED_PROFILE
 
-  echo "Added alias for blt to $DETECTED_PROFILE"
-  echo "Restart your terminal session to use the new command."
+  if cat $DIR/alias >> $DETECTED_PROFILE; then
+    echo "Added alias for blt to $DETECTED_PROFILE"
+    echo "Restart your terminal session to use the new command."
+  else
+    echo "Error: Could not modify $DETECTED_PROFILE."
+  fi
 else
   echo "Could not install blt alias. No profile found. Tried ~/.zshrc, ~/.bashrc, ~/.bash_profile and ~/.profile."
 fi