diff options
author | Rich Trott <rtrott@gmail.com> | 2018-09-26 21:32:37 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-09-27 09:46:22 -0700 |
commit | c9c4550dd48408b7c78a5e1c51b9b19244b66e8c (patch) | |
tree | 25052c50b62c7a02eb933fcc3b35bbb3bcda6f43 /Makefile | |
parent | 3c56b46bfa0f38ce5d5c02cee3134c4b4089c906 (diff) | |
download | android-node-v8-c9c4550dd48408b7c78a5e1c51b9b19244b66e8c.tar.gz android-node-v8-c9c4550dd48408b7c78a5e1c51b9b19244b66e8c.tar.bz2 android-node-v8-c9c4550dd48408b7c78a5e1c51b9b19244b66e8c.zip |
build: move addons message in Makefile
Displaying a message about building addons before building docs can be
confusing when troubleshooting. (This just happened to me.) Move the
message about building addons to just before the step for building
addons.
PR-URL: https://github.com/nodejs/node/pull/23114
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Michaƫl Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Diffstat (limited to 'Makefile')
-rw-r--r-- | Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -278,9 +278,9 @@ jstest: build-addons build-addons-napi ## Runs addon tests and JS tests .PHONY: test # This does not run tests of third-party libraries inside deps. test: all ## Runs default tests, linters, and builds docs. + $(MAKE) -s test-doc @echo "Build the addons before running the tests so the test results" @echo "can be displayed together" - $(MAKE) -s test-doc $(MAKE) -s build-addons $(MAKE) -s build-addons-napi $(MAKE) -s cctest |