diff options
| author | Richard Lau <rlau@redhat.com> | 2021-06-09 18:27:25 +0300 |
|---|---|---|
| committer | Richard Lau <rlau@redhat.com> | 2021-07-20 14:11:34 +0300 |
| commit | 069cf59e56408c490395f5ce44c86b23705596b8 (patch) | |
| tree | a141a3e12bc7c0cc01f86724e2b6b998afd789f0 /Makefile | |
| parent | 443db64eed1c1c4d37e311d423258a8b8f5f49e2 (diff) | |
build: make build-addons errors fail the build
The `build-addons` makefile target runs `tools/doc/addon-verify.js` and
then uses `touch` to update a timestamp file. Unconditionally calling
`touch` was losing the exit code from `tools/doc/addon-verify.js` so
any errors produced by that script were not failing the build.
PR-URL: https://github.com/nodejs/node/pull/38983
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -350,7 +350,7 @@ test/addons/.docbuildstamp: $(DOCBUILDSTAMP_PREREQS) tools/doc/node_modules else \ $(RM) -r test/addons/??_*/; \ [ -x $(NODE) ] && $(NODE) $< || node $< ; \ - touch $@; \ + [ $$? -eq 0 ] && touch $@; \ fi ADDONS_BINDING_GYPS := \ |