[Debian-l10n-commits] r1143 - in /ddtp-dinstall/trunk: Changelog ddtp_i18n_check.sh

[nekral-guest at users.alioth.debian.org]

Author: nekral-guest
Date: Sat Aug  9 03:42:32 2008
New Revision: 1143

URL: http://svn.debian.org/wsvn/?sc=1&rev=1143
Log:
	* ddtp_i18n_check.sh: run with set -u.
	* ddtp_i18n_check.sh: Add "|| false" at the end of the "while
	read". The while will just fail if an internal check "exit 1", but
	the script is not exited. "|| false" makes the script fail (and
	exit) in that case.

Modified:
    ddtp-dinstall/trunk/Changelog
    ddtp-dinstall/trunk/ddtp_i18n_check.sh

Modified: ddtp-dinstall/trunk/Changelog
URL: http://svn.debian.org/wsvn/ddtp-dinstall/trunk/Changelog?rev=1143&op=diff
==============================================================================
--- ddtp-dinstall/trunk/Changelog (original)
+++ ddtp-dinstall/trunk/Changelog Sat Aug  9 03:42:32 2008
@@ -1,3 +1,11 @@
+2008-08-09  Nicolas François  <nicolas.francois at centraliens.net>
+
+	* ddtp_i18n_check.sh: run with set -u.
+	* ddtp_i18n_check.sh: Add "|| false" at the end of the "while
+	read". The while will just fail if an internal check "exit 1", but
+	the script is not exited. "|| false" makes the script fail (and
+	exit) in that case.
+
 2008-08-08  Nicolas François  <nicolas.francois at centraliens.net>
 
 	Thanks for the patience of Felipe Augusto van de Wiel and Michael

Modified: ddtp-dinstall/trunk/ddtp_i18n_check.sh
URL: http://svn.debian.org/wsvn/ddtp-dinstall/trunk/ddtp_i18n_check.sh?rev=1143&op=diff
==============================================================================
--- ddtp-dinstall/trunk/ddtp_i18n_check.sh (original)
+++ ddtp-dinstall/trunk/ddtp_i18n_check.sh Sat Aug  9 03:42:32 2008
@@ -11,7 +11,7 @@
 # On Debian systems, you can find the full text of the license in
 # /usr/share/common-licenses/GPL-2
 
-set -e
+set -eu
 export LC_ALL=C
 
 # This must be defined to either 0 or 1
@@ -303,7 +303,9 @@
 		echo "Neither a file or directory: $f" >&2
 		exit 1
 	fi
-done
+done || false
+# The while will just fail if an internal check "exit 1", but the script
+# is not exited. "|| false" makes the script fail (and exit) in that case.
 
 echo "$dists_parent_dir structure validated successfully ($(date +%c))"