commit 5697b49991a624fffe8228614d2d5e7068ca5067
author Nicholas Piggin <npiggin@gmail.com> Sat Apr 06 22:38:12 2024 +1000
committer Thomas Huth <thuth@redhat.com> Mon Apr 15 13:43:38 2024 +0200
tree 53147697ba99318d6b047124a04222860d281468
parent 20a879fafe5a2b25528fc568c6d51dd6a25783dc

shellcheck: Fix SC2295

  SC2295 (info): Expansions inside ${..} need to be quoted separately,
  otherwise they match as patterns.

Doesn't appear to be a bug since the match string does not include
patterns.

Reviewed-by: Andrew Jones <andrew.jones@linux.dev>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Message-ID: <20240406123833.406488-4-npiggin@gmail.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>

run_tests.sh

diff --git a/run_tests.sh b/run_tests.sh
index 9067e52..116188e 100755
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -99,7 +99,7 @@
         local testname="$1"
         CR=$'\r'
         while read -r line; do
-            line="${line%$CR}"
+            line="${line%"$CR"}"
             case "${line:0:4}" in
                 PASS)
                     echo "ok TEST_NUMBER - ${testname}: ${line#??????}" >&3