diff options
| author | Daniel Bevenius <daniel.bevenius@gmail.com> | 2018-12-12 15:53:19 +0100 |
|---|---|---|
| committer | Rich Trott <rtrott@gmail.com> | 2018-12-15 06:48:07 -0800 |
| commit | e7c3a1b40a0ba8f6cc5364c43e474f8c8de66ef2 (patch) | |
| tree | 19ffffe6562575f2965503c13cd911f9dfe97af4 /Makefile | |
| parent | c9a7088bd80dc190dfc140167998bd004911eef5 (diff) | |
| download | node-new-e7c3a1b40a0ba8f6cc5364c43e474f8c8de66ef2.tar.gz | |
build: make lint-addon-docs run only if needed
Currently, the lint-addon-docs targets recipe will always be run.
This commit makes lint-addon-docs a phony target and adds a new
target named tools/.doclintstamp what will be an actual file,
similar to what the lint-cpp target does.
PR-URL: https://github.com/nodejs/node/pull/24993
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 6 |
1 files changed, 5 insertions, 1 deletions
@@ -1263,10 +1263,14 @@ tools/.cpplintstamp: $(LINT_CPP_FILES) @$(PYTHON) tools/check-imports.py @touch $@ -lint-addon-docs: test/addons/.docbuildstamp +.PHONY: lint-addon-docs +lint-addon-docs: tools/.doclintstamp + +tools/.doclintstamp: test/addons/.docbuildstamp @echo "Running C++ linter on addon docs..." @$(PYTHON) tools/cpplint.py $(CPPLINT_QUIET) --filter=$(ADDON_DOC_LINT_FLAGS) \ $(LINT_CPP_ADDON_DOC_FILES_GLOB) + @touch $@ cpplint: lint-cpp @echo "Please use lint-cpp instead of cpplint" |