build: split CI rules in Makefile
Some CI jobs compile Node and run the tests on different machines. This change enables collaborators to have finer control over what runs on these jobs, such as the exact suites to run. The test-ci rule was split into js and native, to allow for addons to be compiled only on the machines that are going to run them. Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Rod Vagg <rod@vagg.org> PR-URL: https://github.com/nodejs/node/pull/7317
This commit is contained in:
parent
1299c2764e
commit
7cbbec516d
24
Makefile
24
Makefile
@ -184,11 +184,25 @@ test-all: test-build test/gc/node_modules/weak/build/Release/weakref.node
|
||||
test-all-valgrind: test-build
|
||||
$(PYTHON) tools/test.py --mode=debug,release --valgrind
|
||||
|
||||
CI_NATIVE_SUITES := addons
|
||||
CI_JS_SUITES := doctool known_issues message parallel pseudo-tty sequential
|
||||
|
||||
# Build and test addons without building anything else
|
||||
test-ci-native: | test/addons/.buildstamp
|
||||
$(PYTHON) tools/test.py $(PARALLEL_ARGS) -p tap --logfile test.tap \
|
||||
--mode=release --flaky-tests=$(FLAKY_TESTS) \
|
||||
$(TEST_CI_ARGS) $(CI_NATIVE_SUITES)
|
||||
|
||||
# This target should not use a native compiler at all
|
||||
test-ci-js:
|
||||
$(PYTHON) tools/test.py $(PARALLEL_ARGS) -p tap --logfile test.tap \
|
||||
--mode=release --flaky-tests=$(FLAKY_TESTS) \
|
||||
$(TEST_CI_ARGS) $(CI_JS_SUITES)
|
||||
|
||||
test-ci: | build-addons
|
||||
$(PYTHON) tools/test.py $(PARALLEL_ARGS) -p tap --logfile test.tap \
|
||||
--mode=release --flaky-tests=$(FLAKY_TESTS) \
|
||||
$(TEST_CI_ARGS) addons doctool known_issues message pseudo-tty parallel \
|
||||
sequential
|
||||
$(TEST_CI_ARGS) $(CI_NATIVE_SUITES) $(CI_JS_SUITES)
|
||||
|
||||
test-release: test-build
|
||||
$(PYTHON) tools/test.py --mode=release
|
||||
@ -299,9 +313,11 @@ docopen: out/doc/api/all.html
|
||||
docclean:
|
||||
-rm -rf out/doc
|
||||
|
||||
run-ci:
|
||||
build-ci:
|
||||
$(PYTHON) ./configure $(CONFIG_FLAGS)
|
||||
$(MAKE)
|
||||
|
||||
run-ci: build-ci
|
||||
$(MAKE) test-ci
|
||||
|
||||
RAWVER=$(shell $(PYTHON) tools/getnodeversion.py)
|
||||
@ -709,4 +725,4 @@ endif
|
||||
bench-all bench bench-misc bench-array bench-buffer bench-net \
|
||||
bench-http bench-fs bench-tls cctest run-ci test-v8 test-v8-intl \
|
||||
test-v8-benchmarks test-v8-all v8 lint-ci bench-ci jslint-ci doc-only \
|
||||
$(TARBALL)-headers
|
||||
$(TARBALL)-headers test-ci test-ci-native test-ci-js build-ci
|
||||
|
Loading…
x
Reference in New Issue
Block a user