Attention is currently required from: Jakub Czapiga, Julius Werner. Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54072 )
Change subject: tests: code coverage improvements ......................................................................
Patch Set 2:
(3 comments)
File tests/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/54072/comment/83409d1d_fb6716b2 PS1, Line 180: ifeq ($(COV),1)
So there are two options, the one I outlined below, or setting COV=1 if MAKECMDGOALS contains covera […]
I decided to use Patrick's suggestion; if you make coverage-report-tests or clean-coverage-report-tests without COV=1, then the makefile will set COV=1 and call $(MAKE) for that target.
https://review.coreboot.org/c/coreboot/+/54072/comment/9ff8893f_7e7a512e PS1, Line 181: coverage-report-tests:
Not a huge fan of using make targets for all kinds of batch processing, but given how we do that for […]
Done
https://review.coreboot.org/c/coreboot/+/54072/comment/a290cc37_cedc2c88 PS1, Line 187: @echo Please use COV=1 when creating code coverage reports.
simplest solution could be calling `$(MAKE) coverage-report-tests COV=1` here, no?
Done