From 26cf8cc7ed6b2345e970e9832b46afc2e4b1881b Mon Sep 17 00:00:00 2001 From: Clark Boylan Date: Wed, 19 Sep 2012 10:43:15 -0700 Subject: [PATCH] set +e around read in propose_translation_update. The bash read builtin returns a non zero value when it reads EOF. Because the script uses a heredoc read will always read an EOF and always return a non zero value. This non zero value and set -e causes the script to exit. Temporarily set +e to get around this. Change-Id: I2fc2f57ec492ce05089b15bd5b721d27ad4b2be3 Reviewed-on: https://review.openstack.org/13298 Approved: James E. Blair Reviewed-by: James E. Blair Tested-by: Jenkins --- .../files/slave_scripts/propose_translation_update.sh | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/modules/jenkins/files/slave_scripts/propose_translation_update.sh b/modules/jenkins/files/slave_scripts/propose_translation_update.sh index 0b59d51d35..a37965edf7 100755 --- a/modules/jenkins/files/slave_scripts/propose_translation_update.sh +++ b/modules/jenkins/files/slave_scripts/propose_translation_update.sh @@ -15,11 +15,16 @@ change_info=`ssh -p 29418 review.openstack.org gerrit query --current-patch-set previous=`echo "$change_info" | grep "^ number:" | awk '{print $2}'` if [ "x${previous}" != "x" ] ; then change_id=`echo "$change_info" | grep "^change" | awk '{print $2}'` + # read return a non zero value when it reaches EOF. Because we use a + # heredoc here it will always reach EOF and return a nonzero value. + # Disable -e temporarily to get around the read. + set +e read -d '' COMMIT_MSG <