Commit 1ed58ed
build: reduce noise from doc target
The doc target currently echos the complete shell command, which might
produces a lot of output. Using this change the actual commands run
are still shown.
Before:
[ -e tools/doc/node_modules/js-yaml/package.json ] || \
[ -e tools/eslint/node_modules/js-yaml/package.json ] || \
if [ -x ./node ]; then \
cd tools/doc && ../.././node ../.././deps/npm/bin/npm-cli.js
install; \
else \
cd tools/doc && node ../.././deps/npm/bin/npm-cli.js
install; \
fi
[ -x ./node ] && ./node tools/doc/generate.js --node-version=v8.0.0
--format=html --template=doc/template.html doc/api/assert.md >
out/doc/api/assert.html || node tools/doc/generate.js
--node-version=v8.0.0 --format=html --template=doc/template.html
doc/api/assert.md > out/doc/api/assert.html
Input file = doc/api/assert.md
After:
[ -x ./node ] && ./node tools/doc/generate.js --node-version=v8.0.0
--format=html --template=doc/template.html doc/api/assert.md >
out/doc/api/assert.html || node tools/doc/generate.js
--node-version=v8.0.0 --format=html --template=doc/template.html
doc/api/assert.md > out/doc/api/assert.html
Input file = doc/api/assert.md
To verify run:
$ make docclean
$ make doc
PR-URL: #9457
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: James Snell <jasnell@gmail.com>1 parent 093d677 commit 1ed58ed
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -313,7 +313,7 @@ out/doc/%: doc/% | |||
| 313 | 313 | # check if ./node is actually set, else use user pre-installed binary | |
| 314 | 314 | gen-json = tools/doc/generate.js --format=json $< > $@ | |
| 315 | 315 | out/doc/api/%.json: doc/api/%.md | |
| 316 | - [ -e tools/doc/node_modules/js-yaml/package.json ] || \ | ||
| 316 | + @[ -e tools/doc/node_modules/js-yaml/package.json ] || \ | ||
| 317 | 317 | [ -e tools/eslint/node_modules/js-yaml/package.json ] || \ | |
| 318 | 318 | if [ -x $(NODE) ]; then \ | |
| 319 | 319 | cd tools/doc && ../../$(NODE) ../../$(NPM) install; \ | |
@@ -325,7 +325,7 @@ out/doc/api/%.json: doc/api/%.md | |||
| 325 | 325 | # check if ./node is actually set, else use user pre-installed binary | |
| 326 | 326 | gen-html = tools/doc/generate.js --node-version=$(FULLVERSION) --format=html --template=doc/template.html $< > $@ | |
| 327 | 327 | out/doc/api/%.html: doc/api/%.md | |
| 328 | - [ -e tools/doc/node_modules/js-yaml/package.json ] || \ | ||
| 328 | + @[ -e tools/doc/node_modules/js-yaml/package.json ] || \ | ||
| 329 | 329 | [ -e tools/eslint/node_modules/js-yaml/package.json ] || \ | |
| 330 | 330 | if [ -x $(NODE) ]; then \ | |
| 331 | 331 | cd tools/doc && ../../$(NODE) ../../$(NPM) install; \ | |
0 commit comments