Skip to content

Commit

Permalink
Stop quoting multiple parameters together
Browse files Browse the repository at this point in the history
$chroot_opts holds multiple parameters that mustn't be treated as a
single one.  Start treating it as an array in order to satisfy
shellcheck requirements.
  • Loading branch information
tuliom committed Feb 7, 2025
1 parent 68b3e36 commit 007f154
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/fedora-copr-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,8 @@ jobs:
source scripts/functions.sh
[[ -n "${{ matrix.extra_script_file }}" ]] && source ${{ matrix.extra_script_file }}
chroot_opts=$(for c in ${{ env.chroots }}; do echo -n " --chroot $c "; done)
# shellcheck disable=SC2207
chroot_opts=($(for c in ${{ env.chroots }}; do echo -n " --chroot $c "; done))
copr create \
--instructions "$(cat project-instructions.md)" \
Expand All @@ -144,7 +145,7 @@ jobs:
--appstream off \
--delete-after-days 32 \
--module-hotfixes on \
"$chroot_opts" "${{ env.project_today }}"
"${chroot_opts[@]}" "${{ env.project_today }}"
- name: "Enable snapshot_build build condition for all and swig:4.0 module in RHEL 8 build chroots (if any)"
shell: bash -e {0}
Expand Down

0 comments on commit 007f154

Please sign in to comment.