From ea5de4dea46f625d7979601cc2f1be35fafa0310 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 27 Sep 2018 09:50:11 -0700 Subject: [PATCH] build: remove unnecessary Makefile output Remove unnecessary @echo commands from Makefile. These were originally comments but were changed to @echo in 6bc43aeea79. They aren't terribly useful so let's remove them. PR-URL: https://github.com/nodejs/node/pull/23129 Reviewed-By: Luigi Pinca Reviewed-By: Sakthipriyan Vairamani Reviewed-By: Richard Lau Reviewed-By: Colin Ihrig Reviewed-By: Refael Ackermann Reviewed-By: James M Snell --- Makefile | 6 ------ 1 file changed, 6 deletions(-) diff --git a/Makefile b/Makefile index 94f31afa58b80b..a92371573e0c9e 100644 --- a/Makefile +++ b/Makefile @@ -279,8 +279,6 @@ jstest: build-addons build-addons-napi ## Runs addon tests and JS tests # This does not run tests of third-party libraries inside deps. test: all ## Runs default tests, linters, and builds docs. $(MAKE) -s test-doc - @echo "Build the addons before running the tests so the test results" - @echo "can be displayed together" $(MAKE) -s build-addons $(MAKE) -s build-addons-napi $(MAKE) -s cctest @@ -288,8 +286,6 @@ test: all ## Runs default tests, linters, and builds docs. .PHONY: test-only test-only: all ## For a quick test, does not run linter or build docs. - @echo "Build the addons before running the tests so the test results" - @echo "can be displayed together" $(MAKE) build-addons $(MAKE) build-addons-napi $(MAKE) cctest @@ -297,8 +293,6 @@ test-only: all ## For a quick test, does not run linter or build docs. # Used by `make coverage-test` test-cov: all - @echo "Build the addons before running the tests so the test results" - @echo "can be displayed together" $(MAKE) build-addons $(MAKE) build-addons-napi # $(MAKE) cctest