From af81505c00c7d9aee8346bd12db10861f506b3dd Mon Sep 17 00:00:00 2001 From: Peter Simons Date: Thu, 17 Mar 2016 20:57:46 +0100 Subject: [PATCH] wrap-gapps-hook.sh: fix double inclusion guard The simple "return" would not override the non-zero error code set by the preceding test command, therefore aborting scripts running with "set -e". --- pkgs/build-support/setup-hooks/wrap-gapps-hook.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkgs/build-support/setup-hooks/wrap-gapps-hook.sh b/pkgs/build-support/setup-hooks/wrap-gapps-hook.sh index 9e0cd22c1198..3cad1838d260 100644 --- a/pkgs/build-support/setup-hooks/wrap-gapps-hook.sh +++ b/pkgs/build-support/setup-hooks/wrap-gapps-hook.sh @@ -11,7 +11,7 @@ envHooks+=(find_gio_modules) # Note: $gappsWrapperArgs still gets defined even if $dontWrapGApps is set. wrapGAppsHook() { # guard against running multiple times (e.g. due to propagation) - [ -z "$wrapGAppsHookHasRun" ] || return + [ -z "$wrapGAppsHookHasRun" ] || return 0 wrapGAppsHookHasRun=1 if [ -n "$GDK_PIXBUF_MODULE_FILE" ]; then