From a137ed64dcbeef6ed143f57970191c8ab85c419d Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Mon, 17 Jul 2017 00:01:38 -0600 Subject: [PATCH] fix: Declare and assign separately to avoid masking return values. --- plugins/config/docker-args-deploy | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/plugins/config/docker-args-deploy b/plugins/config/docker-args-deploy index df8dee222..367a0f80d 100755 --- a/plugins/config/docker-args-deploy +++ b/plugins/config/docker-args-deploy @@ -5,12 +5,16 @@ source "$PLUGIN_AVAILABLE_PATH/config/functions" config_docker_args() { declare desc="config docker-args plugin trigger" - local trigger="$0 config_docker_args" - local STDIN=$(cat); local APP="$1"; local IMAGE_TAG="$2"; local IMAGE=$(get_deploying_app_image_name "$APP" "$IMAGE_TAG") + declare APP="$1" IMAGE_TAG="$2" + local ENV_ARGS IMAGE STDIN trigger + + IMAGE=$(get_deploying_app_image_name "$APP" "$IMAGE_TAG") + STDIN=$(cat); + trigger="$0 config_docker_args" verify_app_name "$APP" if ! is_image_herokuish_based "$IMAGE"; then - local ENV_ARGS="$(config_export app "$APP" --format docker-args --merged)" + ENV_ARGS="$(config_export app "$APP" --format docker-args --merged)" echo -n "$STDIN $ENV_ARGS" else echo -n "$STDIN"