commit 5f3a4e14e361c073797d89d5475ef34d7737bd23 Author: Isis Lovecruft isis@torproject.org Date: Mon Jun 11 19:16:43 2018 +0000
build: Add `make clippy` option for Rust linting.
Note that clippy is quite loud/obnoxious, as its name would imply. This target isn't meant to run on CI, or even necessarily exit cleanly. It's merely a way for developers to check if they've made any glaring mistakes, or done awkward things they might not have otherwise realised. We *can* configure it to be quieter in the future, but it's probably not worth rabbitholing on.
* CLOSES #22156: https://bugs.torproject.org/22156 --- Makefile.am | 14 ++++++++++++++ 1 file changed, 14 insertions(+)
diff --git a/Makefile.am b/Makefile.am index 58ff9fb3e..c533f577b 100644 --- a/Makefile.am +++ b/Makefile.am @@ -238,6 +238,20 @@ check-typos: echo "You can install the latest version of misspell here: https://github.com/client9/misspell#install"; \ fi
+.PHONY: clippy +clippy: +if USE_RUST + @if test -x "`which cargo-clippy 2>&1;true`"; then \ + echo "Running cargo clippy ..."; \ + echo "Prepare yourself for the onslaught of suggestions ..."; \ + (cd "$(top_srcdir)/src/rust" && cargo clippy); \ + else \ + echo "Tor can use clippy to lint Rust code."; \ + echo "However, it seems that you don't have clippy installed."; \ + echo "You can install the latest version of clippy by following the directions here: https://github.com/rust-lang-nursery/rust-clippy"; \ + fi +endif + .PHONY: check-changes check-changes: if USEPYTHON