Skip to content

Commit 7cfb49a

Browse files
committed
Switch to for-loop without stdin
1 parent 91bdd02 commit 7cfb49a

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

.github/workflows/_test_riscv.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,7 @@ jobs:
6464
# Allows failure in `echo | jq` pipeline to bubble up and fail the workflow
6565
set -o pipefail
6666
67-
# Using process substitution so subprocesses inside the loop (called from .ci/scripts/test_riscv_qemu.sh)
68-
# don't inherit and consume bytes from the jq pipe, corrupting iteration.
69-
while IFS= read -r qemu_cpu_ext; do
67+
for qemu_cpu_ext in $(echo '${{ inputs.qemu-cpu-ext }}' | jq -r '.[]'); do
7068
variant_slug="${qemu_cpu_ext//,/_}"; variant_slug="${variant_slug//=/_}"; variant_slug="${variant_slug:-base}"
7169
build_dir="riscv_test/${{ inputs.model }}${{ inputs.quantize && '_q' || '' }}/${{ inputs.backend }}/${{ inputs.os }}-${{ inputs.arch }}-${variant_slug}"
7270
@@ -105,4 +103,4 @@ jobs:
105103
echo
106104
) >> $GITHUB_STEP_SUMMARY
107105
fi
108-
done < <(echo '${{ inputs.qemu-cpu-ext }}' | jq -r '.[]')
106+
done

0 commit comments

Comments
 (0)