[PATCH] dim: fix shellcheck warning in ask_user
Jani Nikula
jani.nikula at intel.com
Fri Jan 18 12:47:45 UTC 2019
On Wed, 16 Jan 2019, Daniel Vetter <daniel at ffwll.ch> wrote:
> On Wed, Jan 16, 2019 at 01:34:56PM +0200, Jani Nikula wrote:
>> Fix:
>>
>> local prompt="$@ (y/N) "
>> ^-- SC2124: Assigning an array to a string! Assign as
>> array, or use * instead of @ to concatenate.
>>
>> Reported-by: Daniel Vetter <daniel.vetter at ffwll.ch>
>> Fixes: b5403812b664 ("dim: ask the user to create DIM_PREFIX and clone DIM_REPO")
>> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
>
> Reviewed-by: Daniel Vetter <daniel.vetter at ffwll.ch>
Thanks, pushed.
BR,
Jani.
>
>> ---
>> dim | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/dim b/dim
>> index 7a5136c5db26..af6b4ca679a4 100755
>> --- a/dim
>> +++ b/dim
>> @@ -157,7 +157,7 @@ function pause
>>
>> function ask_user
>> {
>> - local prompt="$@ (y/N) "
>> + local prompt="$* (y/N) "
>>
>> read -n 1 -rsp "$prompt"
>> echo
>> --
>> 2.20.1
>>
--
Jani Nikula, Intel Open Source Graphics Center
More information about the dim-tools
mailing list