Makefile: reenable some shellcheck warnings

Bart's patches fixed several shellcheck warnings which we previously
disabled. Reenable them.

Signed-off-by: Omar Sandoval <osandov@fb.com>

diff --git a/Makefile b/Makefile
index 14ba8341dae57e562d70f768d9d613bbcafd0f35..47531e5edb6dc6df7f8c7344f762dc584715fd0c 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -4,18 +4,11 @@ all:
 clean:
        $(MAKE) -C src clean
 
-# SC1090: "Can't follow non-constant source". We use variable sources all over
-# the place.
 # SC2034: "VARIABLE appears unused". All test scripts use this for the test
 # metadata, and many helper functions define global variables.
 # SC2119: "Use foo "$@" if function's $1 should mean script's $1". False
 # positives on helpers like _init_scsi_debug.
-# SC2154: "VARIABLE is referenced but not assigned". False positives on
-# TEST_RUN[foo]=bar.
-# SC2209: "Use var=$(command) to assign output (or quote to assign string)".
-# Warns about DMESG_FILTER=cat, which is not going to confuse anyone who knows
-# how to write shell scripts.
-SHELLCHECK_EXCLUDE := SC1090,SC2034,SC2119,SC2154,SC2209
+SHELLCHECK_EXCLUDE := SC2034,SC2119
 
 check:
        shellcheck -x -e $(SHELLCHECK_EXCLUDE) -f gcc check new common/* tests/*/[0-9]*[0-9]