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

Added a workaround for -Zinstrument-coverage depreciation #7004

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chadbrewbaker
Copy link
Contributor

This works and prints branch coverage. Good for inspecting a single tool, but will need work to use as CI.

@sylvestre
Copy link
Contributor

Would it be possible to bring back the ci too?

@chadbrewbaker
Copy link
Contributor Author

Looks like a failing Windows symlink test. If I recall windows symlink permissions are brittle, there should probably be a system permission check before any tests that require those permissions. https://github.com/uutils/coreutils/actions/runs/12508467521/job/34896676679?pr=7004#step:7:4747

@sylvestre
Copy link
Contributor

Ser #7001 For this

@sylvestre
Copy link
Contributor

btw, not your fault but
cargo llvm-cov --html
fails because ls tests will executed, and llvm-cov will create some profdata files causing the tests to fail.

see the description here:
#6857

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.

2 participants