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

Flip swift.coverage_prefix_map default #1002

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

keith
Copy link
Member

@keith keith commented Feb 14, 2023

Using this flag makes objects, archives, and binaries built with
coverage data entirely hermetic. The only downside is that the coverage
reports generated from these binaries cannot be viewed in Xcode since it
doesn't support handling relative paths. If users want the previous
behavior they can specify --features=-swift.coverage_prefix_map

Using this flag makes objects, archives, and binaries built with
coverage data entirely hermetic. The only downside is that the coverage
reports generated from these binaries cannot be viewed in Xcode since it
doesn't support handling relative paths. If users want the previous
behavior they can specify `--features=-swift.coverage_prefix_map`
@keith
Copy link
Member Author

keith commented Feb 14, 2023

seems like this is the right default, and what we just did for objc bazelbuild/apple_support#190

@keith keith merged commit 1d72aaf into master Feb 15, 2023
@keith keith deleted the ks/flip-swift.coverage_prefix_map-default branch February 15, 2023 18:57
tymurmustafaiev pushed a commit to tymurmustafaiev/rules_swift that referenced this pull request Jul 19, 2023
Using this flag makes objects, archives, and binaries built with
coverage data entirely hermetic. The only downside is that the coverage
reports generated from these binaries cannot be viewed in Xcode since it
doesn't support handling relative paths. If users want the previous
behavior they can specify `--features=-swift.coverage_prefix_map`
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