Skip to content

Commit f37633e

Browse files
avivkelleraduh95
authored andcommitted
build: use $(BUILDTYPE) when cleaning coverage files
PR-URL: #57995 Reviewed-By: Luigi Pinca Reviewed-By: Antoine du Hamel
1 parent d9700fe commit f37633e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Makefile

Lines changed: 2 additions & 2 deletions
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)