[X2Go-Commits] [x2goclient] 80/95: deduplicate.sh: fix wrong return value capture.

git-admin at x2go.org git-admin at x2go.org
Thu Oct 29 03:34:02 CET 2015


This is an automated email from the git hooks/post-receive script.

x2go pushed a commit to branch bugfix/osx
in repository x2goclient.

commit f5b9bf41ad2f4b80af92a316f3d6899688d7d9f0
Author: Mihai Moldovan <ionic at ionic.de>
Date:   Thu Sep 24 03:39:26 2015 +0200

    deduplicate.sh: fix wrong return value capture.
    
    We want to check the return value of the executed function/command, not
    that of "typeset".
---
 debian/changelog |    2 ++
 deduplicate.sh   |    8 ++++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 687a590..abbdd3b 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -147,6 +147,8 @@ x2goclient (4.0.5.1-0x2go1) UNRELEASED; urgency=low
       lazy_canonical_path.
     - deduplicate.sh: remove a bit of noisy debug output, but turn on
       parse_otool_output debugging.
+    - deduplicate.sh: fix wrong return value capture. We want to check the
+      return value of the executed function/command, not that of "typeset".
 
   [ Oleksandr Shneyder ]
   * New upstream release (4.0.5.1):
diff --git a/deduplicate.sh b/deduplicate.sh
index e357529..98f0b26 100755
--- a/deduplicate.sh
+++ b/deduplicate.sh
@@ -170,8 +170,12 @@ done
 for all_entry in "${all_files[@]}"; do
 	typeset otool_out="$(otool -L "${all_entry}")"
 
-	typeset dependencies="$(parse_otool_output "${otool_out}")"
-	echo "parse_otool_output return value: ${?}"
+	# Don't merge the declaration and initialization with the real value assignment.
+	# We need the return value of parse_otool_output(), but running
+	# typeset foo="$(bar)" will give us the return value of typeset, not bar().
+	typeset dependencies=""
+	dependencies="$(parse_otool_output "${otool_out}")"
+
 	if [ "${?}" -eq "0" ]; then
 		typeset line=""
 		while read -r line; do

--
Alioth's /srv/git/code.x2go.org/x2goclient.git//..//_hooks_/post-receive-email on /srv/git/code.x2go.org/x2goclient.git


More information about the x2go-commits mailing list