Commit 63e33ac
build: make doc target quiet
Currently it can be a little difficult to detect errors in the output
from the doc target. This commit suggests reducing the output to make it
easier to identify errors.
PR-URL: #16516
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>1 parent 95a61cb commit 63e33ac
3 files changed
Lines changed: 5 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -542,13 +542,13 @@ doc-only: $(apidocs_html) $(apidocs_json) | |||
| 542 | 542 | doc: $(NODE_EXE) doc-only | |
| 543 | 543 | ||
| 544 | 544 | $(apidoc_dirs): | |
| 545 | - mkdir -p $@ | ||
| 545 | + @mkdir -p $@ | ||
| 546 | 546 | ||
| 547 | 547 | out/doc/api/assets/%: doc/api_assets/% out/doc/api/assets | |
| 548 | - cp $< $@ | ||
| 548 | + @cp $< $@ | ||
| 549 | 549 | ||
| 550 | 550 | out/doc/%: doc/% | |
| 551 | - cp -r $< $@ | ||
| 551 | + @cp -r $< $@ | ||
| 552 | 552 | ||
| 553 | 553 | # check if ./node is actually set, else use user pre-installed binary | |
| 554 | 554 | gen-json = tools/doc/generate.js --format=json $< > $@ | |
@@ -566,11 +566,11 @@ gen-doc = \ | |||
| 566 | 566 | [ -x $(NODE) ] && $(NODE) $(1) || node $(1) | |
| 567 | 567 | ||
| 568 | 568 | out/doc/api/%.json: doc/api/%.md | |
| 569 | - $(call gen-doc, $(gen-json)) | ||
| 569 | + @$(call gen-doc, $(gen-json)) | ||
| 570 | 570 | ||
| 571 | 571 | # check if ./node is actually set, else use user pre-installed binary | |
| 572 | 572 | out/doc/api/%.html: doc/api/%.md | |
| 573 | - $(call gen-doc, $(gen-html)) | ||
| 573 | + @$(call gen-doc, $(gen-html)) | ||
| 574 | 574 | ||
| 575 | 575 | docopen: $(apidocs_html) | |
| 576 | 576 | @$(PYTHON) -mwebbrowser file://$(PWD)/out/doc/api/all.html | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -54,7 +54,6 @@ if (!inputFile) { | |||
| 54 | 54 | throw new Error('No input file specified'); | |
| 55 | 55 | } | |
| 56 | 56 | ||
| 57 | - console.error('Input file = %s', inputFile); | ||
| 58 | 57 | fs.readFile(inputFile, 'utf8', function(er, input) { | |
| 59 | 58 | if (er) throw er; | |
| 60 | 59 | // process the input for @include lines | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -25,7 +25,6 @@ function processIncludes(inputFile, input, cb) { | |||
| 25 | 25 | const includes = input.match(includeExpr); | |
| 26 | 26 | if (includes === null) return cb(null, input); | |
| 27 | 27 | var errState = null; | |
| 28 | - console.error(includes); | ||
| 29 | 28 | var incCount = includes.length; | |
| 30 | 29 | if (incCount === 0) cb(null, input); | |
| 31 | 30 | includes.forEach(function(include) { | |
0 commit comments