Skip to content

Commit 632d7d5

Browse files
rvaggtargos
authored andcommitted
build: skip test-ci doc targets if no crypto
PR-URL: #28747 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
1 parent 5d09c15 commit 632d7d5

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

Makefile

+10-2
Original file line numberDiff line numberDiff line change
@@ -488,7 +488,11 @@ test-all-suites: | clear-stalled test-build bench-addons-build doc-only ## Run a
488488

489489
CI_NATIVE_SUITES ?= addons js-native-api node-api
490490
CI_JS_SUITES ?= default
491-
CI_DOC := doctool
491+
ifeq ($(node_use_openssl), false)
492+
CI_DOC := doctool
493+
else
494+
CI_DOC =
495+
endif
492496

493497
.PHONY: test-ci-native
494498
# Build and test addons without building anything else
@@ -700,7 +704,11 @@ tools/doc/node_modules: tools/doc/package.json
700704
.PHONY: doc-only
701705
doc-only: tools/doc/node_modules \
702706
$(apidoc_dirs) $(apiassets) ## Builds the docs with the local or the global Node.js binary.
703-
@$(MAKE) out/doc/api/all.html out/doc/api/all.json
707+
@if [ "$(shell $(node_use_openssl))" != "true" ]; then \
708+
echo "Skipping doc-only (no crypto)"; \
709+
else \
710+
$(MAKE) out/doc/api/all.html out/doc/api/all.json; \
711+
fi
704712

705713
.PHONY: doc
706714
doc: $(NODE_EXE) doc-only

0 commit comments

Comments
 (0)