Attention is currently required from: Thomas Heijligen, Anastasia Klimchuk.
View Change
1 comment:
Patchset:
Patch Set #1:
This is not the greatest solution, but the only thing I found that worked. Without this we only get coverage for the tests/*.c files.
Also tried : `CFLAGS="-fprofile-dir="(pwd)"/buildcov/cov" meson setup --wipe buildcov -Db_coverage=true`. That puts the gcda files in the right place, but then the `ninja coverage` step fails as the gcno files are still in the old place.
The llvm code coverage does not need this patch.
To view, visit change 69265. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ibba93037831fc0ed526f701cbe360b5a493895d1
Gerrit-Change-Number: 69265
Gerrit-PatchSet: 1
Gerrit-Owner: Evan Benn <evanbenn@google.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: Thomas Heijligen <src@posteo.de>
Gerrit-Attention: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Comment-Date: Mon, 07 Nov 2022 02:13:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment