>>>>> On Thu, 17 Oct 2024, Mike Gilbert wrote:

>>                                      
>> -oUserKnownHostsFile="${known_hosts_file}" \\
>>                                      -oForwardX11=no \\
>>                                      -oClearAllForwardings=yes \\
>> +                                    ${ECVS_SSH_EXTRA_OPTS[*]} \\

> Why use an array if you're going to collapse it using the "*"
> operator? Maybe use "${ECVS_SSH_EXTRA_OPTS[@]}" instead.

This is in a here-document, so the resulting wrapper script will contain
the variable as one word, i.e. "@" won't result in separate words.

However, I now notice that there is a whitespace issue. Patch v2 will
follow.

Ulrich

Attachment: signature.asc
Description: PGP signature

Reply via email to