-
Notifications
You must be signed in to change notification settings - Fork 507
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
[dotnet] Change condition when to include the System.Runtime.InteropServices.NFloat.Internal package reference. #15183
[dotnet] Change condition when to include the System.Runtime.InteropServices.NFloat.Internal package reference. #15183
Conversation
…ervices.NFloat.Internal package reference. '(BundledNETCorePlatformsPackageVersion' will be 7.0 when building with .NET 7 and using the 'net6.0-*' TFM, but we still need the package reference in that case. So change the condition to go off TargetFrameworkVersion instead.
/sudo backport release/6.0.4xx |
Backport Job to branch release/6.0.4xx Created! The magic is happening here |
Hooray! Backport succeeded! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=6234029 for more details. |
💻 [PR Build] Tests on macOS Mac Catalina (10.15) passed 💻✅ All tests on macOS Mac Catalina (10.15) passed. Pipeline on Agent |
📋 [PR Build] API Diff 📋API diff (for current PR)ℹ️ API Diff (from PR only) (please review changes) .NETXamarin vs .NETAPI diff (vs stable)✅ API Diff from stable .NETXamarin vs .NETGenerator diff✅ Generator Diff (no change) Pipeline on Agent XAMBOT-1033.Monterey' |
💻 [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) passed 💻✅ All tests on macOS M1 - Mac Big Sur (11.5) passed. Pipeline on Agent |
❌ [CI Build] Tests failed on VSTS: simulator tests iOS ❌Tests failed on VSTS: simulator tests iOS. Test results19 tests failed, 215 tests passed.Failed tests
Pipeline on Agent XAMBOT-1035.Monterey' |
Test failures are unrelated (https://github.com/xamarin/maccore/issues/2558). |
…m.Runtime.InteropServices.NFloat.Internal package reference. (#15184) 'BundledNETCorePlatformsPackageVersion' will be 7.0 when building with .NET 7 and using the 'net6.0-*' TFM, but we still need the package reference in that case. So change the condition to go off TargetFrameworkVersion instead. Backport of #15183 Co-authored-by: Rolf Bjarne Kvinge <rolf@xamarin.com> Co-authored-by: Alex Soto <alex@alexsoto.me>
'BundledNETCorePlatformsPackageVersion' will be 7.0 when building with .NET 7
and using the 'net6.0-*' TFM, but we still need the package reference in that
case. So change the condition to go off TargetFrameworkVersion instead.