move clean.coverage to coverage.mk

diff --git a/Makefile b/Makefile
index b2fe000cd02e0386447bf0ec5a12cd190a5c4adb..8377f150632656297bbe40e3c3bdc74eaa796c8a 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -563,11 +563,3 @@ endif
 ifneq "$(findstring docker,$(MAKECMDGOALS))" ""
   include scripts/docker/docker.mk
 endif
-
-#
-#  Clean gcov files, too.
-#
-clean: clean.coverage
-.PHONY: clean.coverage
-clean.coverage:
-       @rm -f ${BUILD_DIR}/radiusd.info $(find ${BUILD_DIR} -name "*.gcda" -print)

diff --git a/scripts/build/coverage.mk b/scripts/build/coverage.mk
index 80034e05ae6025e5f69937980c5a0cca15868f2f..13d50a76dd232c2a7333ed600ac78c28ed2e1f16 100644 (file)
--- a/scripts/build/coverage.mk
+++ b/scripts/build/coverage.mk
@@ -1,3 +1,7 @@
+#
+#  Before doing `make coverage`, you should do a `make clean`.
+#
+
 #
 #  The coverage tests require lcov.
 #
@@ -14,10 +18,6 @@
 CFLAGS += -fprofile-arcs -ftest-coverage
 LDFLAGS += -fprofile-instr-generate
 
-#
-#  Before doing `make coverage`, you should do a
-#  `make clean`.
-#
 #  Order is important here.  And the dependencies in the rest of the
 #  makefiles aren't *quite* there to allow for these to be targets.
 #  So we just run them manually one after the other.
@@ -42,3 +42,11 @@ ${BUILD_DIR}/radiusd.info:
 ${BUILD_DIR}/coverage/index.html: ${BUILD_DIR}/radiusd.info
        ${Q}genhtml $< -o $(dir $@) > ${BUILD_DIR}/genhtml.log
        ${Q}echo Please see $@
+
+#
+#  Clean gcov files, too.
+#
+clean: clean.coverage
+.PHONY: clean.coverage
+clean.coverage:
+       @rm -f ${BUILD_DIR}/*.info $(find ${BUILD_DIR} -name "*.gcda" -print)