[main] Update dependencies from mono/linker #50232
Conversation
![dotnet-maestro[bot]](https://avatars.githubusercontent.com/in/16706?s=60&v=4){kind=small}
Microsoft.NET.ILLink.Tasks From Version 6.0.100-preview.2.21172.2 -> To Version 6.0.100-preview.2.21174.1
|
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
Microsoft.NET.ILLink.Tasks From Version 6.0.100-preview.2.21172.2 -> To Version 6.0.100-preview.2.21176.1
Microsoft.NET.ILLink.Tasks From Version 6.0.100-preview.2.21172.2 -> To Version 6.0.100-preview.2.21177.1
| @@ -651,5 +648,9 @@ | |||
| <method name="ApplyUpdateSdb" /> | |||
| </type> | |||
|
|
|||
| <type fullname="System.ValueType"> | |||
| <!-- https://github.com/dotnet/runtime/issues/47909 --> | |||
There was a problem hiding this comment.
I don't understand why this issue is attached to this entry. The issue says
Remove DynamicDependency from ValueType in browser builds
But this entry is unconditionally preserving the constructor on ValueType, which are two different things in my mind.
There was a problem hiding this comment.
I can remove it from the xml if you want. It's only a reminder to remove it once we land final solution for System.Runtime.InteropServices.JavaScript.Runtime dependency.
There was a problem hiding this comment.
Can you explain why we need ValueType.ctor() in the xml?
There was a problem hiding this comment.
We have some tiny sample which managed to not require ValueType hence dropping the required dependency.
There was a problem hiding this comment.
I'd prefer to remove this comment, since it isn't relevant to what is being done here in the xml. I can do that in a follow up PR, so we can merge this to unblock the integration.
ghost
locked as resolved and limited conversation to collaborators
This pull request updates the following dependencies
From https://github.com/mono/linker