-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[release/7.0] Don't try to build PGO tools on platforms with no CoreCLR port. #75003
Conversation
Tagging subscribers to this area: @dotnet/area-infrastructure-libraries Issue DetailsBackport of #74147 to release/7.0 /cc @directhex Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
Tagging subscribers to this area: @directhex Issue DetailsBackport of #74147 to release/7.0 /cc @directhex Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
Re-running the failed CI legs. They were a known, unrelated issue that has already been mitigated. (the max limit of runs was exceeded). |
@mdh1418 or @akoeplinger can we get a code review sign off? Edit: @marek-safar please approve this backport. |
Backport of #74147 to release/7.0
/cc @directhex
Customer Impact
This should only impact customer-supported platforms like s390x and ppc64le, and the backport is by request of the community.
Testing
Customer indicates that the PR in question works for them. CI is fine otherwise on our side.
Risk
Should not impact any of our scenarios.