Commit 5c2ed2e
build: make node-gyp output silent
As it is, node-gyp produces a lot of build related verbose messages.
Latest node-gyp upgrade allows us to specify --silent flag to suppress
those messages. Except for CI, addons build will run silently.
PR-URL: #8990
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Ilkka Myller <ilkka.myller@nodefield.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>1 parent 54def06 commit 5c2ed2e
1 file changed
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -8,6 +8,7 @@ PREFIX ?= /usr/local | |||
| 8 | 8 | FLAKY_TESTS ?= run | |
| 9 | 9 | TEST_CI_ARGS ?= | |
| 10 | 10 | STAGINGSERVER ?= node-www | |
| 11 | + LOGLEVEL ?= silent | ||
| 11 | 12 | OSTYPE := $(shell uname -s | tr '[A-Z]' '[a-z]') | |
| 12 | 13 | ||
| 13 | 14 | ifdef JOBS | |
@@ -163,7 +164,8 @@ test/addons/.buildstamp: config.gypi \ | |||
| 163 | 164 | # Cannot use $(wildcard test/addons/*/) here, it's evaluated before | |
| 164 | 165 | # embedded addons have been generated from the documentation. | |
| 165 | 166 | for dirname in test/addons/*/; do \ | |
| 166 | - $(NODE) deps/npm/node_modules/node-gyp/bin/node-gyp rebuild \ | ||
| 167 | + echo "\nRunning addons test $$PWD/$$dirname" ; \ | ||
| 168 | + $(NODE) deps/npm/node_modules/node-gyp/bin/node-gyp --loglevel=$(LOGLEVEL) rebuild \ | ||
| 167 | 169 | --python="$(PYTHON)" \ | |
| 168 | 170 | --directory="$$PWD/$$dirname" \ | |
| 169 | 171 | --nodedir="$$PWD" || exit 1 ; \ | |
@@ -193,6 +195,7 @@ CI_NATIVE_SUITES := addons | |||
| 193 | 195 | CI_JS_SUITES := doctool inspector known_issues message parallel pseudo-tty sequential | |
| 194 | 196 | ||
| 195 | 197 | # Build and test addons without building anything else | |
| 198 | + test-ci-native: LOGLEVEL := info | ||
| 196 | 199 | test-ci-native: | test/addons/.buildstamp | |
| 197 | 200 | $(PYTHON) tools/test.py $(PARALLEL_ARGS) -p tap --logfile test.tap \ | |
| 198 | 201 | --mode=release --flaky-tests=$(FLAKY_TESTS) \ | |
@@ -204,6 +207,7 @@ test-ci-js: | |||
| 204 | 207 | --mode=release --flaky-tests=$(FLAKY_TESTS) \ | |
| 205 | 208 | $(TEST_CI_ARGS) $(CI_JS_SUITES) | |
| 206 | 209 | ||
| 210 | + test-ci: LOGLEVEL := info | ||
| 207 | 211 | test-ci: | build-addons | |
| 208 | 212 | out/Release/cctest --gtest_output=tap:cctest.tap | |
| 209 | 213 | $(PYTHON) tools/test.py $(PARALLEL_ARGS) -p tap --logfile test.tap \ | |
0 commit comments