Subject: Re: [PATCH] ls-tree: test for the regression in 9c4d58ff2c3

From: Junio C Hamano <[email-masked]>
Date: Thu, 02 Jun 2022 10:48:09 -0700

Johannes Schindelin <[email-masked]> writes:

>> +test_ls_tree_format_mode_output () {
>> +	local opts=$1 &&
>
> This line does not quote `$1`, but...
>
> ... this line passes a first argument that contains spaces. Hence the
> tests fail in CI:
> https://github.com/git/git/runs/6703333447?check_suite_focus=true

We had a portability discussion not so in distant past on "local".

  https://lore.kernel.org/git/[email-masked]/

The conclusion from the thread IIRC is

 * Very safe

	local var; var=$1

 * Probably OK

	local var="$1"

 * Not portable

	local var=$1

Thanks.