From: Zdenek Kabelac Date: Fri, 14 Jul 2017 18:16:54 +0000 (+0200) Subject: tests: aux.sh fix double quote X-Git-Tag: v2_02_173~41 X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=ac0945bc4a14dfe8def578f9132e7261f254a279;p=lvm2.git tests: aux.sh fix double quote Patch 72a58ce4b0f68f6e45ae30606fe5af21caa42b43 was wronly placing double quotes around this variable which we want to pass expanded, as it's just set of 'space' device args ATM. TODO consider using array[@] to make this cleaner. Add shellcheck directive to skip warning here --- diff --git a/test/lib/aux.sh b/test/lib/aux.sh index a0e05ffbf..9a4868958 100644 --- a/test/lib/aux.sh +++ b/test/lib/aux.sh @@ -224,8 +224,9 @@ prepare_lvmetad() { # Default debug is "-l all" and could be override # by setting LVM_TEST_LVMETAD_DEBUG_OPTS before calling inittest. echo -n "## preparing lvmetad..." + # shellcheck disable=SC2086 $run_valgrind lvmetad -f "$@" -s "$TESTDIR/lvmetad.socket" \ - "${LVM_TEST_LVMETAD_DEBUG_OPTS--l all}" & + ${LVM_TEST_LVMETAD_DEBUG_OPTS--l all} & echo $! > LOCAL_LVMETAD for i in {1..100} ; do test "$i" -eq 100 && die "Startup of lvmetad is too slow."