diff mbox series

Makefile: Use '--ignore-errors unused' with lcov

Message ID 20240802182943.179900-2-arbab@linux.ibm.com
State Accepted
Headers show
Series Makefile: Use '--ignore-errors unused' with lcov | expand

Commit Message

Reza Arbab Aug. 2, 2024, 6:29 p.m. UTC
We are a bit overzealous in specifying arguments to 'lcov -r', listing
files (via wildcard) that are not actually in the tracefile.

This is harmless, but will cause newer lcov to generate an error message
of type 'unused'. Reduce this error to a warning.

Signed-off-by: Reza Arbab <arbab@linux.ibm.com>
---
 Makefile.main | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Reza Arbab Sept. 10, 2024, 6:52 p.m. UTC | #1
On Fri, Aug 02, 2024 at 01:29:42PM -0500, Reza Arbab wrote:
>We are a bit overzealous in specifying arguments to 'lcov -r', listing
>files (via wildcard) that are not actually in the tracefile.
>
>This is harmless, but will cause newer lcov to generate an error message
>of type 'unused'. Reduce this error to a warning.

Applied to master.
diff mbox series

Patch

diff --git a/Makefile.main b/Makefile.main
index 5240d71b1457..d25d1c9cd954 100644
--- a/Makefile.main
+++ b/Makefile.main
@@ -413,7 +413,7 @@  skiboot.info: coverage external/pflash/pflash.info external/gard/gard.info
 	lcov -q -r $@ 'external/gard/*' -o $@
 	lcov -q -a $@ -a external/pflash/pflash.info -o $@
 	lcov -q -a $@ -a external/gard/gard.info -o $@
-	lcov -q -r $@ $(LCOV_EXCLUDE) -o $@ --rc lcov_branch_coverage=1
+	lcov -q -r $@ $(LCOV_EXCLUDE) -o $@ --rc lcov_branch_coverage=1 --ignore-errors unused
 
 doc:
 	make -C doc html