Skip to content
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 #7630

Merged
merged 4 commits into from Dec 16, 2022

Conversation

dotnet-maestro[bot]
Copy link
Contributor

This pull request updates the following dependencies

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

  • Coherency Updates:
    • Microsoft.NET.ILLink.Tasks: from 7.0.100-1.22564.1 to 8.0.100-1.22609.1 (parent: Microsoft.Dotnet.Sdk.Internal)
    • Microsoft.NETCore.App.Ref: from 8.0.0-alpha.1.22559.2 to 8.0.0-alpha.1.22605.1 (parent: Microsoft.Dotnet.Sdk.Internal)
    • Microsoft.NET.Workload.Emscripten.net7.Manifest-8.0.100: from 8.0.0-alpha.1.22554.1 to 8.0.0-alpha.1.22558.2 (parent: Microsoft.NETCore.App.Ref)

From https://github.com/dotnet/installer

  • Subscription: 0ba14476-1d1a-452a-3b67-08da9690aef0
  • Build: 20221211.1
  • Date Produced: December 12, 2022 3:00:17 AM UTC
  • Commit: 167a4ed5b457e404bd6eadb4b0fd307000aefd57
  • Branch: refs/heads/main

…221211.1

Microsoft.Dotnet.Sdk.Internal
 From Version 8.0.100-alpha.1.22602.5 -> To Version 8.0.100-alpha.1.22611.1

Dependency coherency updates

Microsoft.NET.ILLink.Tasks,Microsoft.NETCore.App.Ref,Microsoft.NET.Workload.Emscripten.net7.Manifest-8.0.100
 From Version 7.0.100-1.22564.1 -> To Version 8.0.100-1.22609.1 (parent: Microsoft.Dotnet.Sdk.Internal
Comment on lines +40 to 48
"lib/arm64-v8a/libmono-component-marshal-ilgen.so": {
"Size": 93552
},
"lib/arm64-v8a/libmonodroid.so": {
"Size": 434632
"Size": 434704
},
"lib/arm64-v8a/libmonosgen-2.0.so": {
"Size": 3076080
"Size": 3064488
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@naricc does libmono-component-marshal-ilgen.so look like the expected file size? Does it seem like libmonosgen-2.0.so should get smaller? We lost 11kb for one file, but gained 93kb in a new one?

Copy link

@naricc naricc Dec 13, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That looks like about the expected file size for libmono-component-marshal-ilgen.so. But I would expect the size of libmono-sgen-2.0.so to shrink more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@naricc should I file an issue in dotnet/runtime? It's not a large app size regression, but maybe someone should check we don't have duplicate methods in there or something.

Copy link

@naricc naricc Dec 17, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, file an issue, I will look into it.

Context: dotnet/linker#3124 (comment)

Android projects are class libraries, and this current triggers:

    ILLink error IL1034: Root assembly 'MyAndroidApp, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null' does not have entry point.
@jonathanpeppers jonathanpeppers merged commit 23fe98b into main Dec 16, 2022
@jonathanpeppers jonathanpeppers deleted the darc-main-5ce75da7-0011-4d8b-af12-c8d7f2bb59e2 branch December 16, 2022 21:58
grendello added a commit to grendello/xamarin-android that referenced this pull request Jan 4, 2023
* main:
  [Xamarin.Android.Build.Tasks] use %(TrimmerRootAssembly.RootMode) All (xamarin#7651)
  Bump to dotnet/installer@47a747f 8.0.100-alpha.1.22616.7 (xamarin#7647)
  Bump to dotnet/installer@167a4ed 8.0.100-alpha.1.22611.1 (xamarin#7630)
  Bump to xamarin/java.interop@f8d77fa (xamarin#7638)
  [ci] Fix Designer test paths (xamarin#7635)
  [Xamarin.Android.Build.Tasks] perf improvements for LlvmIrGenerator (xamarin#7626)
  [Xamarin.Android.Build.Tasks] AutoImport `*.webp` files (xamarin#7601)
  Localized file check-in by OneLocBuild Task (xamarin#7632)
  Bump $(ProductVersion) to 13.2.99
  Bump to xamarin/monodroid@c0c933b7 (xamarin#7633)
  [Xamarin.Android.Build.Tasks] Fix aapt_rules.txt corruption (xamarin#7587)
  [Xamarin.Android.Build.Tasks] Add XA1031 error (xamarin#7448)
  Bump to xamarin/java.interop@149d70f (xamarin#7625)
  [CODEOWNERS] More updates to CODEOWNERS (xamarin#7628)
  [Xamarin.Android.Build.Tasks] avoid `File.Exists()` check (xamarin#7621)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants