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

Suppress warnings for remote Clang targets #7423

Merged
merged 2 commits into from Mar 27, 2024

Conversation

xedin
Copy link
Member

@xedin xedin commented Mar 25, 2024

This mirrors behavior of remote swift targets which suppress warnings as well.

@xedin
Copy link
Member Author

xedin commented Mar 25, 2024

@MaxDesiatov what would be the best way to test this? I couldn't find any tests with -suppress-warnings flag in the suite :/

@xedin
Copy link
Member Author

xedin commented Mar 25, 2024

This is actually a draft, "package" doesn't exist for clang targets yet, I'm about open a PR to introduce that.

@xedin
Copy link
Member Author

xedin commented Mar 25, 2024

I think I figured it out - I can extend ClangTargetBuildDescriptionTests to check that by forming a remote package dependency.

@MaxDesiatov
Copy link
Member

@MaxDesiatov what would be the best way to test this? I couldn't find any tests with -suppress-warnings flag in the suite :/

This looks like something that belongs to BuildPlanTests or maybe worth creating a new ClangTargetBuildPlanTests as the existing BuildPlanTests is too big.

@MaxDesiatov MaxDesiatov added build system Changes to interactions with build systems diagnostics labels Mar 26, 2024
@MaxDesiatov
Copy link
Member

maybe worth creating a new ClangTargetBuildPlanTests as the existing BuildPlanTests is too big.

Turns out we already have ClangTargetBuildDescriptionTests.swift.

@MaxDesiatov MaxDesiatov added the needs tests This change needs test coverage label Mar 26, 2024
This mirrors behavior of remote swift targets which
suppress warnings as well.
Remote in this case means local or remote source control or registry.
@xedin xedin force-pushed the suppress-warnings-for-remote-clang-targets branch from 9ff1739 to 08efdf5 Compare March 27, 2024 17:27
@xedin
Copy link
Member Author

xedin commented Mar 27, 2024

@swift-ci please test

@xedin
Copy link
Member Author

xedin commented Mar 27, 2024

@swift-ci please test Windows platform

@MaxDesiatov
Copy link
Member

@swift-ci test windows

@xedin xedin merged commit c4a3af1 into apple:main Mar 27, 2024
5 checks passed
@xedin xedin removed the needs tests This change needs test coverage label Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system Changes to interactions with build systems diagnostics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants