Skip to content

Commit 5ea1492

Browse files
mhdawsonrvagg
authored andcommitted
src: fix code coverage cleanup
In #17987 which updated the location of the code coverage patches I missed a few changes needed to properly clean up for code coverage. Add these. PR-URL: #18081 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Jon Moss <me@jonathanmoss.me> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 0d2a720 commit 5ea1492

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
@@ -137,7 +137,7 @@ check: test
137137
coverage-clean:
138138
if [ -d lib_ ]; then $(RM) -r lib; mv lib_ lib; fi
139139
$(RM) -r node_modules
140-
$(RM) -r gcovr testing
140+
$(RM) -r gcovr build
141141
$(RM) -r out/$(BUILDTYPE)/.coverage
142142
$(RM) -r .cov_tmp
143143
$(RM) out/$(BUILDTYPE)/obj.target/node/{src,gen}/*.gcda
@@ -162,7 +162,7 @@ coverage-build: all
162162
$(NODE) ./deps/npm install nyc --no-save --no-package-lock; fi
163163
if [ ! -d gcovr ]; then git clone --depth=1 \
164164
--single-branch git://github.com/gcovr/gcovr.git; fi
165-
if [ ! -d testing ]; then git clone --depth=1 \
165+
if [ ! -d build ]; then git clone --depth=1 \
166166
--single-branch https://github.com/nodejs/build.git; fi
167167
if [ ! -f gcovr/scripts/gcovr.orig ]; then \
168168
(cd gcovr && patch -N -p1 < \

0 commit comments

Comments
 (0)