-
Notifications
You must be signed in to change notification settings - Fork 534
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
[main] Update dependencies from dotnet/installer #6029
[main] Update dependencies from dotnet/installer #6029
Conversation
…210615.23 Microsoft.Dotnet.Sdk.Internal From Version 6.0.100-preview.6.21313.2 -> To Version 6.0.100-preview.6.21315.23
/cc @rolfbjarne triggered this a few minutes ago, and still not seeing anything Preview 7 coming through. |
…210618.1 Microsoft.Dotnet.Sdk.Internal From Version 6.0.100-preview.6.21313.2 -> To Version 6.0.100-preview.6.21318.1
Looks like we'll get P7 when this PR is merged: https://github.com/dotnet/installer/pull/10856/files |
…210618.5 Microsoft.Dotnet.Sdk.Internal From Version 6.0.100-preview.6.21313.2 -> To Version 6.0.100-preview.6.21318.5 Dependency coherency updates Microsoft.NET.ILLink.Tasks,Microsoft.NETCore.App.Ref From Version 6.0.100-preview.6.21304.2 -> To Version 6.0.100-preview.6.21317.4 (parent: Microsoft.Dotnet.Sdk.Internal
Ok, seeing Preview 7 builds from dotnet/runtime now. We should branch before we merge this, but I might wait until Monday to see if everything will be Preview 7. |
I branched xamarin-macios, and we're getting P6 bumps there: xamarin/xamarin-macios#11989 |
…210621.2 Microsoft.Dotnet.Sdk.Internal From Version 6.0.100-preview.6.21313.2 -> To Version 6.0.100-preview.7.21321.2 Dependency coherency updates Microsoft.NET.ILLink.Tasks,Microsoft.NETCore.App.Ref From Version 6.0.100-preview.6.21304.2 -> To Version 6.0.100-preview.6.21317.4 (parent: Microsoft.Dotnet.Sdk.Internal
I just branched for Preview 6 as well: https://github.com/xamarin/xamarin-android/tree/release/6.0.1xx-preview6 |
Context: dotnet/runtime#53811 dotnet/runtime#53811 is now solved, so we can remove the workaround.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/cc @steveisok it seems like a native library grew a bit in size here:
689,448 lib/arm64-v8a/libmonosgen-2.0.so
Is this unexpected? ~689K seems like a lot of growth.
The other one, is a small enough difference to ignore:
12,288 lib/arm64-v8a/libSystem.Security.Cryptography.Native.Android.so
Actually seeing a size difference here, too: #6034
|
Context: dotnet/msbuild#6609 It seems that properties are missing from MSBuild logs, ignoring part of this test for now.
Context: dotnet/templating#3325 Context: https://github.com/dotnet/templating/wiki/Reference-for-template.json#content-manipulation In the current bump, `dotnet new android` + `dotnet build` fails with: Resources\mipmap-xxxhdpi\ic_launcher_round.png error APT2000: failed reading from input: PNG chunk type 49444154 is too large: chunk length is 10836 but chunk starts at byte 45/8187. The .NET templating system has a 'copyOnly' mode for files that do not need any text replaced. This is a performance feature, but it also happens to workaround this issue. We should be doing this on `.png` files *anyway*, as we don't need potential "text" to be replaced. I will probably need to make similar changes in xamarin/xamarin-macios and dotnet/maui.
This pull request updates the following dependencies
From https://github.com/dotnet/installer
Coherency Updates
The following updates ensure that dependencies with a CoherentParentDependency
attribute were produced in a build used as input to the parent dependency's build.
See Dependency Description Format