Re: [PATCH 1/2] legacy stash: fix "rudimentary backport of -q"

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 11, 2019, 07:27 UTC
Message-ID: <xmqqlg1lkh11.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <f76c7cb66c43a6c4bf712516949a867b09710103.1551972571.git.gitgitgadget@gmail.com>

"Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com> writes:

Show 8 quoted lines

> This can be seen very clearly when passing arguments with special
> characters, like
>
> 	git stash -- ':(glob)**/*.txt'
>
> Since this is exactly what we want to test in the next commit (where we
> fix this very incantation with the built-in stash), let's fix the legacy
> scripted version of `git stash` first.

Thanks. I agree that these eval are evaluating one iteration too much.

Show 32 quoted lines

>
> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> ---
>  git-legacy-stash.sh | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/git-legacy-stash.sh b/git-legacy-stash.sh
> index 8a8c4a9270..f60e9b3e87 100755
> --- a/git-legacy-stash.sh
> +++ b/git-legacy-stash.sh
> @@ -86,17 +86,17 @@ maybe_quiet () {
>  		shift
>  		if test -n "$GIT_QUIET"
>  		then
> -			eval "$@" 2>/dev/null
> +			"$@" 2>/dev/null
>  		else
> -			eval "$@"
> +			"$@"
>  		fi
>  		;;
>  	*)
>  		if test -n "$GIT_QUIET"
>  		then
> -			eval "$@" >/dev/null 2>&1
> +			"$@" >/dev/null 2>&1
>  		else
> -			eval "$@"
> +			"$@"
>  		fi
>  		;;
>  	esac