1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

build: make lint-ci work properly on Linux make

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/19746
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
This commit is contained in:
Rod Vagg 2018-04-02 19:37:11 +10:00
parent 67bbc84185
commit c5928ab631
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Makefile
View File

@ -1133,7 +1133,8 @@ lint-js-ci:
jslint-ci: lint-js-ci jslint-ci: lint-js-ci
@echo "Please use lint-js-ci instead of jslint-ci" @echo "Please use lint-js-ci instead of jslint-ci"
LINT_CPP_ADDON_DOC_FILES = $(wildcard test/addons/??_*/*.cc test/addons/??_*/*.h) LINT_CPP_ADDON_DOC_FILES_GLOB = test/addons/??_*/*.cc test/addons/??_*/*.h
LINT_CPP_ADDON_DOC_FILES = $(wildcard $(LINT_CPP_ADDON_DOC_FILES_GLOB))
LINT_CPP_EXCLUDE ?= LINT_CPP_EXCLUDE ?=
LINT_CPP_EXCLUDE += src/node_root_certs.h LINT_CPP_EXCLUDE += src/node_root_certs.h
LINT_CPP_EXCLUDE += $(LINT_CPP_ADDON_DOC_FILES) LINT_CPP_EXCLUDE += $(LINT_CPP_ADDON_DOC_FILES)
@ -1176,7 +1177,7 @@ tools/.cpplintstamp: $(LINT_CPP_FILES)
lint-addon-docs: test/addons/.docbuildstamp lint-addon-docs: test/addons/.docbuildstamp
@echo "Running C++ linter on addon docs..." @echo "Running C++ linter on addon docs..."
@$(PYTHON) tools/cpplint.py --filter=$(ADDON_DOC_LINT_FLAGS) $(LINT_CPP_ADDON_DOC_FILES) @$(PYTHON) tools/cpplint.py --filter=$(ADDON_DOC_LINT_FLAGS) $(LINT_CPP_ADDON_DOC_FILES_GLOB)
cpplint: lint-cpp cpplint: lint-cpp
@echo "Please use lint-cpp instead of cpplint" @echo "Please use lint-cpp instead of cpplint"