Skip to content

Commit e61aa0c

Browse files
authored
build: use $(BUILDTYPE) when cleaning coverage files
PR-URL: nodejs#57995 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
1 parent a7cbb90 commit e61aa0c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Makefile

+2-2
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ coverage-clean: ## Remove coverage artifacts.
238238
$(RM) -r node_modules
239239
$(RM) -r gcovr
240240
$(RM) -r coverage/tmp
241-
@if [ -d "out/Release/obj.target" ]; then \
241+
@if [ -d "out/$(BUILDTYPE)/obj.target" ]; then \
242242
$(FIND) out/$(BUILDTYPE)/obj.target \( -name "*.gcda" -o -name "*.gcno" \) \
243243
-type f | xargs $(RM); \
244244
fi
@@ -265,7 +265,7 @@ coverage-build-js: ## Build JavaScript coverage files.
265265

266266
.PHONY: coverage-test
267267
coverage-test: coverage-build ## Run the tests and generate a coverage report.
268-
@if [ -d "out/Release/obj.target" ]; then \
268+
@if [ -d "out/$(BUILDTYPE)/obj.target" ]; then \
269269
$(FIND) out/$(BUILDTYPE)/obj.target -name "*.gcda" -type f | xargs $(RM); \
270270
fi
271271
-NODE_V8_COVERAGE=coverage/tmp \

0 commit comments

Comments
 (0)