From c091bd89cf58c19cdd49cb79f031effd1123e896 Mon Sep 17 00:00:00 2001 From: "Yann E. MORIN" Date: Wed, 5 Jun 2024 19:53:19 +0200 Subject: [PATCH] support/download/cvs: fix shellcheck Signed-off-by: Yann E. MORIN Signed-off-by: Arnout Vandecappelle (cherry picked from commit 84309d7071de9c54a3633967eb2abaa75b8dc9f9) Signed-off-by: Peter Korsgaard --- .checkpackageignore | 1 - support/download/cvs | 3 ++- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.checkpackageignore b/.checkpackageignore index d7681a4db6..051287841e 100644 --- a/.checkpackageignore +++ b/.checkpackageignore @@ -1439,7 +1439,6 @@ support/dependencies/check-host-xzcat.sh Shellcheck support/dependencies/dependencies.sh Shellcheck support/download/bzr ConsecutiveEmptyLines Shellcheck support/download/cargo-post-process Shellcheck -support/download/cvs Shellcheck support/download/dl-wrapper Shellcheck support/download/file Shellcheck support/download/go-post-process Shellcheck diff --git a/support/download/cvs b/support/download/cvs index 0c079e2403..b771caf280 100755 --- a/support/download/cvs +++ b/support/download/cvs @@ -40,12 +40,13 @@ shift $((OPTIND-1)) # Get rid of our options # 10 minutes should do the trick. _cvs() { if [ -z "${quiet}" ]; then - printf '%s ' timeout 10m ${CVS} "${@}"; printf '\n' + printf '%s ' timeout 10m "${CVS}" "${@}"; printf '\n' fi _plain_cvs "$@" } # Note: please keep command below aligned with what is printed above _plain_cvs() { + # shellcheck disable=SC2086 # We really want word splitting in CVS eval timeout 10m ${CVS} "${@}" }