diff options
| author | Gibson Fahnestock <gibfahn@gmail.com> | 2017-09-17 00:43:54 +0100 |
|---|---|---|
| committer | Ruben Bridgewater <ruben@bridgewater.de> | 2017-09-21 19:23:36 -0300 |
| commit | 66e45b821a3f70cfde7c6e2feda7e17a8607ae9c (patch) | |
| tree | 115649a2527e7dfd8987989587355ecb5525d7a4 /Makefile | |
| parent | 78339e25111cee826debf0e864c7f8d894f3288c (diff) | |
| download | node-new-66e45b821a3f70cfde7c6e2feda7e17a8607ae9c.tar.gz | |
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 |