Skip to content

Commit 1ed58ed

Browse files
danbevMyles Borins
authored and
Myles Borins
committed
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 <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: James Snell <[email protected]>
1 parent 093d677 commit 1ed58ed

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Makefile

+2-2
Original file line numberDiff line numberDiff line change
@@ -313,7 +313,7 @@ out/doc/%: doc/%
313313
# check if ./node is actually set, else use user pre-installed binary
314314
gen-json = tools/doc/generate.js --format=json $< > $@
315315
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 ] || \
317317
[ -e tools/eslint/node_modules/js-yaml/package.json ] || \
318318
if [ -x $(NODE) ]; then \
319319
cd tools/doc && ../../$(NODE) ../../$(NPM) install; \
@@ -325,7 +325,7 @@ out/doc/api/%.json: doc/api/%.md
325325
# check if ./node is actually set, else use user pre-installed binary
326326
gen-html = tools/doc/generate.js --node-version=$(FULLVERSION) --format=html --template=doc/template.html $< > $@
327327
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 ] || \
329329
[ -e tools/eslint/node_modules/js-yaml/package.json ] || \
330330
if [ -x $(NODE) ]; then \
331331
cd tools/doc && ../../$(NODE) ../../$(NPM) install; \

0 commit comments

Comments
 (0)