build: make linter targets silent

thefourtheye · evanlucas · commit e6be4b951a37 · 2017-05-02T12:46:11.000-05:00
The linter targets are printing the commands they execute on screen. This patch reduces the noise by not printing the commands. PR-URL: #12423 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/Makefile b/Makefile
@@ -809,10 +809,12 @@ bench: bench-net bench-http bench-fs bench-tls
 bench-ci: bench
 
 jslint:
+	@echo "Running JS linter..."
 	$(NODE) tools/eslint/bin/eslint.js --cache --rulesdir=tools/eslint-rules \
 	  benchmark lib test tools
 
 jslint-ci:
+	@echo "Running JS linter..."
 	$(NODE) tools/jslint.js $(PARALLEL_ARGS) -f tap -o test-eslint.tap \
 		benchmark lib test tools
 
@@ -836,12 +838,13 @@ CPPLINT_FILES = $(filter-out $(CPPLINT_EXCLUDE), $(wildcard \
 	))
 
 cpplint:
+	@echo "Running C++ linter..."
 	@$(PYTHON) tools/cpplint.py $(CPPLINT_FILES)
 	@$(PYTHON) tools/check-imports.py
 
 ifneq ("","$(wildcard tools/eslint/lib/eslint.js)")
 lint:
-	EXIT_STATUS=0 ; \
+	@EXIT_STATUS=0 ; \
 	$(MAKE) jslint || EXIT_STATUS=$$? ; \
 	$(MAKE) cpplint || EXIT_STATUS=$$? ; \
 	exit $$EXIT_STATUS
