^-- SC2153: Possible misspelling: BASELINE may not be assigned, but baseline is.
BASELINE is defined inside patches/config, but this confuses shellcheck. According to https://github.com/koalaman/shellcheck/wiki/SC2153 the message doesn't appear for guarded references where if variable is defined a default is used. Maybe we should pick a better default like the git HEAD ?! Cc: Jani Nikula <[email protected]> Cc: Daniel Vetter <[email protected]> Cc: Joonas Lahtinen <[email protected]> Signed-off-by: Rodrigo Vivi <[email protected]> --- qf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/qf b/qf index 059c8e5afe1d..a42ef4779cab 100755 --- a/qf +++ b/qf @@ -85,7 +85,7 @@ function repo_check # allow-detached fi # shellcheck source=/dev/null - baseline=$(source patches/config ; echo $BASELINE) + baseline=$(source patches/config ; echo ${BASELINE:-}) if [[ $(git rev-parse HEAD) != "$baseline" ]] ; then echo Baseline commit doesn\'t match with quilt config @@ -233,7 +233,7 @@ function qf_checkout # error code 2 means no patches removed, which is ok quiet_pop_all -f # shellcheck source=/dev/null - baseline=$(source patches/config ; echo $BASELINE) + baseline=$(source patches/config ; echo ${BASELINE:-}) checkout_baseline quilt push -a -q > /dev/null echo Now at $(quilt top) -- 2.13.2 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
