Send patches - preferably formatted by git format-patch - to patches at archlinux32 dot org.
summaryrefslogtreecommitdiff
path: root/scripts/libmakepkg/executable.sh.in
diff options
context:
space:
mode:
authorEli Schwartz <eschwartz@archlinux.org>2018-11-27 23:00:36 -0500
committerAllan McRae <allan@archlinux.org>2018-12-04 17:22:27 +1000
commit9f1b735d76d04665d6324e723e163a52ca27567c (patch)
treefd58de354872e9343d90e97ba8630417ab017db7 /scripts/libmakepkg/executable.sh.in
parent65e09705d3ad781fd41cf1996f2b60e2e2c1975b (diff)
libmakepkg/executable: don't rely on scoped value of $ret to flag outcomes
Elsewhere, we return 1 if a library dropin fails, and when running functions in a loop, we use `|| ret=1` to preserve scope. This ensures the return value of the function remains useful in isolation. Do the same thing here as well. Drop trivial function which wraps a dropin that also uses $ret, since it's no longer needed. Signed-off-by: Eli Schwartz <eschwartz@archlinux.org> Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'scripts/libmakepkg/executable.sh.in')
-rw-r--r--scripts/libmakepkg/executable.sh.in2
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/libmakepkg/executable.sh.in b/scripts/libmakepkg/executable.sh.in
index 68c48038..92031d43 100644
--- a/scripts/libmakepkg/executable.sh.in
+++ b/scripts/libmakepkg/executable.sh.in
@@ -38,7 +38,7 @@ check_software() {
local ret=0
for func in ${executable_functions[@]}; do
- $func
+ $func || ret=1
done
return $ret