Skip to content

Update rust_stdlib_filegroup to detect profiler_builtins#3958

Merged
UebelAndre merged 2 commits intobazelbuild:mainfrom
UebelAndre:coverage
Apr 16, 2026
Merged

Update rust_stdlib_filegroup to detect profiler_builtins#3958
UebelAndre merged 2 commits intobazelbuild:mainfrom
UebelAndre:coverage

Conversation

@UebelAndre
Copy link
Copy Markdown
Collaborator

This change adds a coverage_enabled to rust_toolchain that should be used to identify if the toolchain supports coverage. This would allow repositories which use MUSL to run bazel coverage //... without running into a linker error. The downside is that they will get no coverage but this would already be the case due to the linker error. More details on rust-lang/rust#79556

@UebelAndre UebelAndre changed the title Update rust_stdlib_filegroup to detect profiler-builtins Update rust_stdlib_filegroup to detect profiler_builtins Apr 9, 2026
@UebelAndre UebelAndre requested a review from krasimirgg April 9, 2026 15:00
@UebelAndre UebelAndre marked this pull request as ready for review April 9, 2026 15:00
@UebelAndre
Copy link
Copy Markdown
Collaborator Author

Friendly ping @krasimirgg

Copy link
Copy Markdown
Collaborator

@krasimirgg krasimirgg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Looks good!

@UebelAndre UebelAndre enabled auto-merge April 16, 2026 12:56
@UebelAndre UebelAndre added this pull request to the merge queue Apr 16, 2026
Merged via the queue into bazelbuild:main with commit eb280fa Apr 16, 2026
3 checks passed
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