[main] Update dependencies from dotnet/runtime#28584
Conversation
…1013.13 Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.8.0 , VS.Redist.Common.NetCore.TargetingPack.x64.8.0 From Version 8.0.0-alpha.1.22512.8 -> To Version 8.0.0-alpha.1.22513.13
Notification for subscribed users from https://github.com/dotnet/runtime:@dnr-codeflow Action requested: Please take a look at this failing automated dependency-flow pull request's checks; failures may be related to changes which originated in your repo.
|
eng/Version.Details.xml
Outdated
| <SourceBuild RepoName="linker" ManagedOnly="true" /> | ||
| </Dependency> | ||
| <Dependency Name="Microsoft.DotNet.ILCompiler" Version="8.0.0-alpha.1.22512.8"> | ||
| <Dependency Name="Microsoft.DotNet.ILCompiler" Version="8.0.0-alpha.1.22513.13"> |
There was a problem hiding this comment.
@jkotas, the ILCompiler package version being updated is 8.x. But the tests are trying to use binaries from 7.0.0-rc.1.22426.10 package, while picking Microsoft.NETCore.Native.targets from 8.0.100-ci and failing due to missing new files we recently added in 8.x https://helix.dot.net/api/2019-06-17/jobs/2c21a5b3-5b77-4883-ae23-f6a7b4c70305/workitems/Microsoft.NET.Publish.Tests.dll.2/console.
Should they use the same version for Microsoft.NETCore.Native.targets and the binaries? This mismatch of version is unsustainable.
There was a problem hiding this comment.
@LakshanF Could you please look into this?
It is the kind of problem with versioning Microsoft.NETCore.Native.targets that I was worried about.
…1014.9 Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.8.0 , VS.Redist.Common.NetCore.TargetingPack.x64.8.0 From Version 8.0.0-alpha.1.22512.8 -> To Version 8.0.0-alpha.1.22514.9
…1015.3 Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.8.0 , VS.Redist.Common.NetCore.TargetingPack.x64.8.0 From Version 8.0.0-alpha.1.22512.8 -> To Version 8.0.0-alpha.1.22515.3
…1016.1 Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.8.0 , VS.Redist.Common.NetCore.TargetingPack.x64.8.0 From Version 8.0.0-alpha.1.22512.8 -> To Version 8.0.0-alpha.1.22516.1
…1017.2 Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.8.0 , VS.Redist.Common.NetCore.TargetingPack.x64.8.0 From Version 8.0.0-alpha.1.22512.8 -> To Version 8.0.0-alpha.1.22517.2
…1018.11 Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.8.0 , VS.Redist.Common.NetCore.TargetingPack.x64.8.0 From Version 8.0.0-alpha.1.22512.8 -> To Version 8.0.0-alpha.1.22518.11
|
@LakshanF can you take a look? Thanks |
Instead of disabling all aot tests on linux, we can set dotnet/runtime#75468 will fix this issue permanently for different major versions (we basically need to defer the import of |
…1019.7 Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.8.0 , VS.Redist.Common.NetCore.TargetingPack.x64.8.0 From Version 8.0.0-alpha.1.22512.8 -> To Version 8.0.0-alpha.1.22519.7
…s://github.com/dotnet/sdk into darc-main-ea02237a-f937-463c-a5a2-ef46b098f7a2
src/Tests/Microsoft.NET.Publish.Tests/GivenThatWeWantToPublishAnAotApp.cs
Outdated
Show resolved
Hide resolved
|
There is another AOT test failing: |
…AnAotApp.cs Co-authored-by: Jan Kotas <jkotas@microsoft.com>
| if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows)) | ||
| { | ||
| // Workaround for https://github.com/dotnet/runtime/issues/75468. Delete once the issue is fixed. | ||
| testProject.AdditionalProperties["LinkStandardCPlusPlusLibrary"] = "true"; |
There was a problem hiding this comment.
Can we put this in the targets file instead? That way it will actually work, instead of just passing this test.
This pull request updates the following dependencies
From https://github.com/dotnet/runtime