Skip to content

Commit

Permalink
Add lcov to Bookworm slave container (#18544)
Browse files Browse the repository at this point in the history
The gcovr package used to have lcov as a runtime dependency. However,
that was removed in Debian bug 987818. However, autoconf's code coverage
macro file expects lcov and genhtml to be present.

It's possible that lcov and genhtml are actually not needed for
generating the report, and that gcovr is enough. For now, though, just
add lcov (which includes genhtml) into the slave container.

Signed-off-by: Saikrishna Arcot <[email protected]>
  • Loading branch information
saiarcot895 authored and pull[bot] committed Aug 17, 2024
1 parent a323370 commit 6ab6243
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions sonic-slave-bookworm/Dockerfile.j2
Original file line number Diff line number Diff line change
Expand Up @@ -258,6 +258,7 @@ RUN apt-get update && apt-get install -y \
python3-nose \
python3-venv \
gcovr \
lcov \
python3-pytest-cov \
python3-pytest-cov \
python3-parse \
Expand Down

0 comments on commit 6ab6243

Please sign in to comment.