Makefile: Hide coverage helper targets if COV is not set
We only check COV=1 in `make coverage` before. This will result in
deleting all files in the current working directory if we accidentally
tried `make coverage_init`.
Hide coverage_init and coverage_html if COV is not set.
BUG=none
BRANCH=none
TEST=make coverage_init and see the error message
TEST=make clean && COV=1 make coverage -j
Signed-off-by: Hsuan Ting Chen <roccochen@chromium.org>
Change-Id: I2d22b211b3d9e59d12faa0ab3d1c2ca8433598f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3747419
Reviewed-by: Yu-Ping Wu <yupingso@chromium.org>
diff --git a/Makefile b/Makefile
index afb6e91..a3566d4 100644
--- a/Makefile
+++ b/Makefile
@@ -1348,6 +1348,11 @@
${Q}echo -e "\nruntests: \E[32;1mALL TESTS PASSED SUCCESSFULLY!\E[0;m\n"
# Code coverage
+.PHONY: coverage
+ifeq ($(filter-out 0,${COV}),)
+coverage:
+ $(error Build coverage like this: make clean && COV=1 make coverage)
+else
.PHONY: coverage_init
coverage_init: install_for_test
rm -f ${COV_INFO}*
@@ -1366,11 +1371,6 @@
lcov -e ${COV_INFO}.nostub '${SRCDIR}/firmware/*' \
-o ${COV_INFO}.firmware
-.PHONY: coverage
-ifeq ($(filter-out 0,${COV}),)
-coverage:
- $(error Build coverage like this: make clean && COV=1 make coverage)
-else
coverage: coverage_init runtests coverage_html
endif
