Exit immediately in case of error, add quotes around paths

diff --git a/bin/i18n.sh b/bin/i18n.sh
index 580248770e490fd2cec70d8a566f40bd3393423f..6ff5beb69ced4af161f95e0979832c5995c41cde 100755 (executable)
--- a/bin/i18n.sh
+++ b/bin/i18n.sh
@@ -18,9 +18,11 @@
 # along with WeeChat.org.  If not, see <https://www.gnu.org/licenses/>.
 #
 
-DIR=$(cd $(dirname "$0"); pwd)
+set -o errexit
 
-cd $DIR/../weechat || exit 1
+DIR=$(cd "$(dirname "$0")"; pwd)
+
+cd "${DIR}/../weechat"
 
 chmod 644 locale/*/LC_MESSAGES/django.po
 
@@ -29,7 +31,7 @@ django-admin makemessages -a
 
 # edit locale if given as argument
 if [ $# -gt 0 ]; then
-    $EDITOR locale/$1/LC_MESSAGES/django.po || exit 1
+    "$EDITOR" "locale/$1/LC_MESSAGES/django.po"
 fi
 
 # compile messages

diff --git a/bin/install.sh b/bin/install.sh
index af62f8b19b5f3fd142e7a5ee6cafa3135228fe50..554ee631ff958682410008b519658af291f085a9 100755 (executable)
--- a/bin/install.sh
+++ b/bin/install.sh
@@ -18,21 +18,23 @@
 # along with WeeChat.org.  If not, see <https://www.gnu.org/licenses/>.
 #
 
-DIR=$(cd $(dirname "$0"); pwd)
+set -o errexit
 
-cd $DIR/.. || exit 1
+DIR=$(cd "$(dirname "$0")"; pwd)
+
+cd "$DIR/.."
 
 echo ""
 echo "--- Compiling messages"
-./manage.py compilemessages || exit 1
+./manage.py compilemessages
 
 echo ""
 echo "--- Creating database"
-./manage.py migrate --run-syncdb || exit 1
+./manage.py migrate --run-syncdb
 
 echo ""
 echo "--- Loading fixtures in database"
-./manage.py loaddata ./weechat/fixtures/*.json || exit 1
+./manage.py loaddata ./weechat/fixtures/*.json
 
 echo ""
 echo "--- Install OK!"