[RFC kvm-unit-tests PATCH v2 09/14] shellcheck: Fix SC2145
Nicholas Piggin
npiggin at gmail.com
Sat Apr 6 05:38:18 PDT 2024
SC2145 (error): Argument mixes string and array. Use * or separate
argument.
Shouldn't be a bug, since the preceding string ends with a space and
there aren't any succeeding strings.
Reviewed-by: Andrew Jones <andrew.jones at linux.dev>
Signed-off-by: Nicholas Piggin <npiggin at gmail.com>
---
arm/efi/run | 2 +-
riscv/efi/run | 2 +-
scripts/mkstandalone.sh | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arm/efi/run b/arm/efi/run
index f07a6e55c..cf6d34b0b 100755
--- a/arm/efi/run
+++ b/arm/efi/run
@@ -87,7 +87,7 @@ uefi_shell_run()
if [ "$EFI_DIRECT" = "y" ]; then
$TEST_DIR/run \
$KERNEL_NAME \
- -append "$(basename $KERNEL_NAME) ${cmd_args[@]}" \
+ -append "$(basename $KERNEL_NAME) ${cmd_args[*]}" \
-bios "$EFI_UEFI" \
"${qemu_args[@]}"
else
diff --git a/riscv/efi/run b/riscv/efi/run
index 982b8b9c4..cce068694 100755
--- a/riscv/efi/run
+++ b/riscv/efi/run
@@ -97,7 +97,7 @@ if [ "$EFI_DIRECT" = "y" ]; then
fi
$TEST_DIR/run \
$KERNEL_NAME \
- -append "$(basename $KERNEL_NAME) ${cmd_args[@]}" \
+ -append "$(basename $KERNEL_NAME) ${cmd_args[*]}" \
-machine pflash0=pflash0 \
-blockdev node-name=pflash0,driver=file,read-only=on,filename="$EFI_UEFI" \
"${qemu_args[@]}"
diff --git a/scripts/mkstandalone.sh b/scripts/mkstandalone.sh
index 86c7e5498..756647f29 100755
--- a/scripts/mkstandalone.sh
+++ b/scripts/mkstandalone.sh
@@ -76,7 +76,7 @@ generate_test ()
cat scripts/runtime.bash
- echo "run ${args[@]}"
+ echo "run ${args[*]}"
}
function mkstandalone()
--
2.43.0
More information about the kvm-riscv
mailing list