Commit 7135822
build: make tools/doc/node_modules non-phony
This commit makes the target tools/doc/node_modules a non-phony target
and also adds tools/doc/package.json as a prerequisite to it to avoid
running it unnecessary. This is currently causing the target
test/addons/.docbuildstamp to be always be executed as it has
tools/doc/node_modules as a prerequisite.
PR-URL: #22189
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Sam Ruby <rubys@intertwingly.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>1 parent 3baeb82 commit 7135822
1 file changed
Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1066,8 +1066,7 @@ lint-md-build: tools/remark-cli/node_modules \ | |||
| 1066 | 1066 | tools/doc/node_modules \ | |
| 1067 | 1067 | tools/remark-preset-lint-node/node_modules | |
| 1068 | 1068 | ||
| 1069 | - .PHONY: tools/doc/node_modules | ||
| 1070 | - tools/doc/node_modules: | ||
| 1069 | + tools/doc/node_modules: tools/doc/package.json | ||
| 1071 | 1070 | ifeq ($(node_use_openssl),true) | |
| 1072 | 1071 | cd tools/doc && $(call available-node,$(run-npm-install)) | |
| 1073 | 1072 | else | |
0 commit comments