Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[gcov] install lcov for gcov in docker-soniv-vs.gz #7377

Closed
wants to merge 1 commit into from
Closed

[gcov] install lcov for gcov in docker-soniv-vs.gz #7377

wants to merge 1 commit into from

Conversation

pettershao-ragilenetworks
Copy link
Contributor

This is for gcov swss repo, need be merge before gcov swss repo.

Why I did it

This lcov tool need install first for html report generation before gcov repo pr be submitted.

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

This is for gcov swss repo, need be merge before gcov swss repo.
@pettershao-ragilenetworks
Copy link
Contributor Author

@lguohan we plan to submit our solution in swss repo this week, since we plan to use test_zzgcov.py(will submit later) in pytest to trigger gcda collection and html report generation, need to install lcov tool in docker-sonic-vs first, please help review and merge this, thanks!

@lguohan
Copy link
Collaborator

lguohan commented Apr 20, 2021

can you share the full design first, and let's review the full design before we starting to review any pr.

@pettershao-ragilenetworks
Copy link
Contributor Author

hi @lguohan, sonic-net/SONiC#642 has updated, help move on, thanks, it is the tool needed by gcov report gereration in vs docker.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants