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

Switch to using stock gcovr 5.2 (#726) #731

Merged
merged 3 commits into from
Jan 3, 2023

Conversation

liushilongbuaa
Copy link
Contributor

  • Switch to using stock gcovr 5.2

The custom branch we were using previously has since been deleted. That branch appears to have some fix for searching for the source file (for a gcda file) recursively. I don't know if it's needed or not today, but using the stock gcovr 5.2 (from the official repo) appears to work.

Signed-off-by: Saikrishna Arcot [email protected]

saiarcot895 and others added 3 commits December 29, 2022 18:06
* Switch to using stock gcovr 5.2

The custom branch we were using previously has since been deleted. That
branch appears to have some fix for searching for the source file (for a
gcda file) recursively. I don't know if it's needed or not today, but
using the stock gcovr 5.2 (from the official repo) appears to work.

Signed-off-by: Saikrishna Arcot <[email protected]>
)

Since sonic-db-cli depends on libswsscommon, we could not simply only purge libswsscommon, so we purge both together.

The build error is as below

Step 8/19 : RUN dpkg --purge libswsscommon
 ---> Running in ed8132d457e9
dpkg: dependency problems prevent removal of libswsscommon:
 sonic-db-cli depends on libswsscommon.

dpkg: error processing package libswsscommon (--purge):
 dependency problems - not removing
Errors were encountered while processing:
 libswsscommon
@liushilongbuaa liushilongbuaa marked this pull request as ready for review January 3, 2023 09:36
@liushilongbuaa liushilongbuaa merged commit 27e5228 into sonic-net:202111 Jan 3, 2023
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.

4 participants