Skip to content

[5.7] suppress warnings from remote dependencies #5608

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

Merged
merged 1 commit into from
Jun 18, 2022

Conversation

tomerd
Copy link
Contributor

@tomerd tomerd commented Jun 18, 2022

5.7 cherry pick of #5605

motivation: warnings from remote dependencies are not actionable and make the use of -warnings-as-errors difficult

changes:
* wire up the ResolvedPackage to SwiftTargetBuildDescription
* pass -suppress-warnings for remote dependencies
* add unit test

rdar://94473684
@tomerd tomerd added ready Author believes the PR is ready to be merged & any feedback has been addressed 5.7 labels Jun 18, 2022
@tomerd
Copy link
Contributor Author

tomerd commented Jun 18, 2022

@swift-ci smoke test

@tomerd tomerd changed the title suppress warnings from remote dependencies [5.7] suppress warnings from remote dependencies Jun 18, 2022
@tomerd tomerd merged commit 796b7ff into swiftlang:release/5.7 Jun 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Author believes the PR is ready to be merged & any feedback has been addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants