diff options
author | Rich Trott <rtrott@gmail.com> | 2018-07-20 14:14:03 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-07-23 14:28:23 -0700 |
commit | b38b8d3b9d02dba33f8f80e53f838220729811a8 (patch) | |
tree | 51182befee9b91649ec3900495ee4e4fa0a01ed2 /Makefile | |
parent | c57e11c2f6a5353b1be0e056322f327e736cf623 (diff) | |
download | android-node-v8-b38b8d3b9d02dba33f8f80e53f838220729811a8.tar.gz android-node-v8-b38b8d3b9d02dba33f8f80e53f838220729811a8.tar.bz2 android-node-v8-b38b8d3b9d02dba33f8f80e53f838220729811a8.zip |
build: remove redundant Makefile target
The only target that uses the
`tools/doc/node_modules/js-yaml/package.json` target is `doc-only`. As a
result of a recent change, it has `tools/doc/node_modules` as a
prerequisite, which does the exact same npm install as
`tools/doc/node_modules/js-yaml/package.json`. Remove
`tools/doc/node_modules/js-yaml/package.json` as unnecessary.
PR-URL: https://github.com/nodejs/node/pull/21915
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'Makefile')
-rw-r--r-- | Makefile | 8 |
1 files changed, 0 insertions, 8 deletions
@@ -612,11 +612,6 @@ apiassets = $(subst api_assets,api/assets,$(addprefix out/,$(wildcard doc/api_as .PHONY: doc-only doc-only: tools/doc/node_modules \ $(apidoc_dirs) $(apiassets) ## Builds the docs with the local or the global Node.js binary. -# If it's a source tarball, assets are already in doc/api/assets, -# no need to install anything, we have already copied the docs over - if [ ! -d doc/api/assets ]; then \ - $(MAKE) tools/doc/node_modules/js-yaml/package.json; \ - fi; @$(MAKE) out/doc/api/all.html out/doc/api/all.json .PHONY: doc @@ -655,9 +650,6 @@ available-node = \ run-npm-install = $(PWD)/$(NPM) install --production --no-package-lock run-npm-ci = $(PWD)/$(NPM) ci -tools/doc/node_modules/js-yaml/package.json: - cd tools/doc && $(call available-node,$(run-npm-install)) - gen-json = tools/doc/generate.js --format=json $< > $@ gen-html = tools/doc/generate.js --node-version=$(FULLVERSION) --format=html \ --analytics=$(DOCS_ANALYTICS) $< > $@ |