[X2Go-Commits] [x2goclient] 64/219: deduplicate.sh: fix parse_otool_output: we want to return failure only iff any of the strings are present in otool's output and actually handle the failure string *ARRAY* correctly as that.
git-admin at x2go.org
git-admin at x2go.org
Thu Sep 22 04:36:20 CEST 2016
This is an automated email from the git hooks/post-receive script.
x2go pushed a commit to branch bugfix/osx
in repository x2goclient.
commit 9a147ac7394cf7dff8dbd45789dd1b73431fd0db
Author: Mihai Moldovan <ionic at ionic.de>
Date: Thu Aug 27 04:53:44 2015 +0200
deduplicate.sh: fix parse_otool_output: we want to return failure only iff any of the strings are present in otool's output and actually handle the failure string *ARRAY* correctly as that.
---
debian/changelog | 3 +++
deduplicate.sh | 6 +++---
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/debian/changelog b/debian/changelog
index 0492e65..35f45da 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -118,6 +118,9 @@ x2goclient (4.0.5.3-0x2go1) UNRELEASED; urgency=medium
"sparse" arrays correctly.
- deduplicate.sh: quote "${arr[@]}" correctly to not suddenly force word
splitting.
+ - deduplicate.sh: fix parse_otool_output: we want to return failure only
+ iff any of the strings are present in otool's output and actually handle
+ the failure string *ARRAY* correctly as that.
-- X2Go Release Manager <git-admin at x2go.org> Mon, 19 Sep 2016 09:07:07 +0200
diff --git a/deduplicate.sh b/deduplicate.sh
index 33c4628..939eb55 100755
--- a/deduplicate.sh
+++ b/deduplicate.sh
@@ -15,9 +15,9 @@ parse_otool_output() {
typeset raw_output="${@}"
typeset fail_str=""
- for fail_str in ${otool_fail_str}; do
- if ! echo "${raw_output}" | grep -q "${fail_str}"; then
- exit 1
+ for fail_str in "${otool_fail_str[@]}"; do
+ if echo "${raw_output}" | grep -q "${fail_str}"; then
+ return 1
fi
done
--
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