Skip to content
This repository was archived by the owner on Aug 31, 2018. It is now read-only.

Commit e4a898a

Browse files
danbevQard
authored andcommitted
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: nodejs/node#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 daf7adb commit e4a898a

File tree

3 files changed

+5
-7
lines changed

3 files changed

+5
-7
lines changed

Makefile

+5-5
Original file line numberDiff line numberDiff line change
@@ -545,13 +545,13 @@ doc-only: $(apidocs_html) $(apidocs_json)
545545
doc: $(NODE_EXE) doc-only
546546

547547
$(apidoc_dirs):
548-
mkdir -p $@
548+
@mkdir -p $@
549549

550550
out/doc/api/assets/%: doc/api_assets/% out/doc/api/assets
551-
cp $< $@
551+
@cp $< $@
552552

553553
out/doc/%: doc/%
554-
cp -r $< $@
554+
@cp -r $< $@
555555

556556
# check if ./node is actually set, else use user pre-installed binary
557557
gen-json = tools/doc/generate.js --format=json $< > $@
@@ -569,11 +569,11 @@ gen-doc = \
569569
[ -x $(NODE) ] && $(NODE) $(1) || node $(1)
570570

571571
out/doc/api/%.json: doc/api/%.md
572-
$(call gen-doc, $(gen-json))
572+
@$(call gen-doc, $(gen-json))
573573

574574
# check if ./node is actually set, else use user pre-installed binary
575575
out/doc/api/%.html: doc/api/%.md
576-
$(call gen-doc, $(gen-html))
576+
@$(call gen-doc, $(gen-html))
577577

578578
docopen: $(apidocs_html)
579579
@$(PYTHON) -mwebbrowser file://$(PWD)/out/doc/api/all.html

tools/doc/generate.js

-1
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ if (!inputFile) {
5454
throw new Error('No input file specified');
5555
}
5656

57-
console.error('Input file = %s', inputFile);
5857
fs.readFile(inputFile, 'utf8', function(er, input) {
5958
if (er) throw er;
6059
// process the input for @include lines

tools/doc/preprocess.js

-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ function processIncludes(inputFile, input, cb) {
2525
const includes = input.match(includeExpr);
2626
if (includes === null) return cb(null, input);
2727
var errState = null;
28-
console.error(includes);
2928
var incCount = includes.length;
3029
if (incCount === 0) cb(null, input);
3130
includes.forEach(function(include) {

0 commit comments

Comments
 (0)