From 76c7918e5ed5a8dcd06b625e12c6d71032d551ff Mon Sep 17 00:00:00 2001 From: Robert Jefe Lindstaedt Date: Fri, 20 May 2016 23:16:47 +0200 Subject: [PATCH] doc: make doc-only -> fallback to user binary After the #3888 it was not possible to "make doc-only" in some situations. This now fallsback to any installed node version and throws "node not found" in error case. Ref: https://github.com/nodejs/node/pull/3888 PR-URL: https://github.com/nodejs/node/pull/6906 Reviewed-By: Myles Borins Reviewed-By: Anna Henningsen --- Makefile | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 82a38aca0795b4..28f8f808808123 100644 --- a/Makefile +++ b/Makefile @@ -272,11 +272,15 @@ out/doc/api/assets/%: doc/api_assets/% out/doc/api/assets/ out/doc/%: doc/% cp -r $< $@ +# check if ./node is actually set, else use user pre-installed binary +gen-json = tools/doc/generate.js --format=json $< > $@ out/doc/api/%.json: doc/api/%.md - $(NODE) tools/doc/generate.js --format=json $< > $@ + [ -x $(NODE) ] && $(NODE) $(gen-json) || node $(gen-json) +# check if ./node is actually set, else use user pre-installed binary +gen-html = tools/doc/generate.js --node-version=$(FULLVERSION) --format=html --template=doc/template.html $< > $@ out/doc/api/%.html: doc/api/%.md - $(NODE) tools/doc/generate.js --node-version=$(FULLVERSION) --format=html --template=doc/template.html $< > $@ + [ -x $(NODE) ] && $(NODE) $(gen-html) || node $(gen-html) docopen: out/doc/api/all.html -google-chrome out/doc/api/all.html