From: Alan T. DeKok <aland@freeradius.org>
Date: Sat, 22 Feb 2025 13:46:57 +0000 (-0500)
Subject: move clean.coverage to coverage.mk
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e119889bd7beca61f26020e53757d49acd488cba;p=thirdparty%2Ffreeradius-server.git

move clean.coverage to coverage.mk
---

diff --git a/Makefile b/Makefile
index b2fe000cd02..8377f150632 100644
--- 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 80034e05ae6..13d50a76dd2 100644
--- 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)