diff options
| author | Gibson Fahnestock <gibfahn@gmail.com> | 2017-09-17 02:43:54 +0300 |
|---|---|---|
| committer | James M Snell <jasnell@gmail.com> | 2017-09-25 21:10:39 +0300 |
| commit | 9685b9ff2fa1455177b24e6c24271a358d86b1bb (patch) | |
| tree | fd6fde04241ae326973dcd41a504c9b9b76a8112 /Makefile | |
| parent | 58b9495c428c9a7e4856b4662f2da090523800ac (diff) | |
build: don't fail `make test` on source tarballs
Tries to achieve the same effect as
https://github.com/nodejs/node/pull/13658 without breaking source
tarballs. Presumably if `tools/eslint` wasn't there at all, people
would notice in the PR review!
PR-URL: https://github.com/nodejs/node/pull/15441
Fixes: https://github.com/nodejs/node/issues/14513
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 3 |
1 files changed, 1 insertions, 2 deletions
@@ -922,7 +922,7 @@ cpplint: @$(PYTHON) tools/cpplint.py $(CPPLINT_FILES) @$(PYTHON) tools/check-imports.py -ifneq ("","$(wildcard tools/eslint/bin/eslint.js)") +ifneq ("","$(wildcard tools/eslint/)") lint: @EXIT_STATUS=0 ; \ $(MAKE) jslint || EXIT_STATUS=$$? ; \ @@ -943,7 +943,6 @@ lint: @echo "Linting is not available through the source tarball." @echo "Use the git repo instead:" \ "$ git clone https://github.com/nodejs/node.git" - exit 1 lint-ci: lint endif |