]> granicus.if.org Git - zfs/commitdiff
Add codecheck make target
authorMatthew Ahrens <mahrens@delphix.com>
Thu, 4 Oct 2018 20:10:10 +0000 (13:10 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 4 Oct 2018 20:10:10 +0000 (13:10 -0700)
We'd like to have tooling that verifies code style, while ignoring the
commit message.  For example, code does not need to be signed off in
order to be tested.  Current workarounds are to run `git checkstyle` and
ignore the commit message errors, or to run `make cstyle shellcheck
flake8 mancheck testscheck`, and make sure that list stays updated.

Solution is to add a new make target, `codecheck` which does all the
code checks.  `checkstyle` is now simply `codecheck` + `commitcheck`.

Reviewed-by: Giuseppe Di Natale <guss80@gmail.com>
Reviewed-by: George Melikov <mail@gmelikov.ru>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed by: Pavel Zakharov <pavel.zakharov@delphix.com>
Signed-off-by: Matthew Ahrens <mahrens@delphix.com>
Closes #7985

Makefile.am

index f2cd574a9ce18d9308cc832d6ff359415e405262..61bf9e235090fbca7391e5874a5bd90b85174e32 100644 (file)
@@ -61,7 +61,9 @@ install-data-hook:
        ln -fs zfs.release spl.release
 endif
 
-checkstyle: cstyle shellcheck flake8 commitcheck mancheck testscheck
+codecheck: cstyle shellcheck flake8 mancheck testscheck
+
+checkstyle: codecheck commitcheck
 
 commitcheck:
        @if git rev-parse --git-dir > /dev/null 2>&1; then \