Skip to content

Commit

Permalink
tools: don't build node for doc build
Browse files Browse the repository at this point in the history
Building node binary is not necessary for doc builds.

Refer: nodejs#9436 (comment)
  • Loading branch information
thefourtheye committed Nov 17, 2016
1 parent c5678d3 commit b71edf4
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -303,8 +303,7 @@ apidoc_dirs = out/doc out/doc/api/ out/doc/api/assets

apiassets = $(subst api_assets,api/assets,$(addprefix out/,$(wildcard doc/api_assets/*)))

doc-only: $(apidocs_html) $(apidocs_json)
doc: $(NODE_EXE) doc-only
doc: $(apidocs_html) $(apidocs_json)

$(apidoc_dirs):
mkdir -p $@
Expand Down Expand Up @@ -750,5 +749,5 @@ endif
blog blogclean tar binary release-only bench-http-simple bench-idle \
bench-all bench bench-misc bench-array bench-buffer bench-net \
bench-http bench-fs bench-tls cctest run-ci test-v8 test-v8-intl \
test-v8-benchmarks test-v8-all v8 lint-ci bench-ci jslint-ci doc-only \
test-v8-benchmarks test-v8-all v8 lint-ci bench-ci jslint-ci \
$(TARBALL)-headers test-ci test-ci-native test-ci-js build-ci

0 comments on commit b71edf4

Please sign in to comment.