From c10e990267e33c3c59b967d5a838e01a21158ae3 Mon Sep 17 00:00:00 2001
From: Jonathan Pobst <jonathan.pobst@microsoft.com>
Date: Mon, 21 Oct 2024 13:16:41 -1000
Subject: [PATCH] Updates needed to build on `net9.0-android` RTM.

---
 source/AndroidXProject.cshtml                 |  3 ++-
 source/_PackageLevelCustomizations.cshtml     |  9 +++++++
 .../paging-common-jvm/Transforms/Metadata.xml |  8 ++++++
 .../work-runtime/Transforms/Metadata.xml      |  1 +
 .../protobuf-lite/Transforms/Metadata.xml     |  8 ++++++
 .../kotlin-stdlib/Transforms/Metadata.xml     | 25 ++++++++++++++++++-
 6 files changed, 52 insertions(+), 2 deletions(-)

diff --git a/source/AndroidXProject.cshtml b/source/AndroidXProject.cshtml
index dc5b9e974..10ec5b39c 100644
--- a/source/AndroidXProject.cshtml
+++ b/source/AndroidXProject.cshtml
@@ -22,8 +22,9 @@
        - CS0809: Obsolete member 'member' overrides non-obsolete member 'member'
        - CS1572: XML comment has a param tag for '', but there is no parameter by that name
        - XAOBS001: While this member is 'public', Google considers it internal API and reserves the right to modify or delete it in the future. Use at your own risk. 
+       - NU1605: Detected package downgrade
     -->
-    <NoWarn>0618;0109;0114;0628;0108;0809;1572;XAOBS001</NoWarn>
+    <NoWarn>0618;0109;0114;0628;0108;0809;1572;XAOBS001;NU1605</NoWarn>
 
 @if (Model.AllowPrereleaseDependencies) {
     <!-- Allow this package to have prerelease dependencies -->
diff --git a/source/_PackageLevelCustomizations.cshtml b/source/_PackageLevelCustomizations.cshtml
index 4f48b1bc7..f63328583 100644
--- a/source/_PackageLevelCustomizations.cshtml
+++ b/source/_PackageLevelCustomizations.cshtml
@@ -55,3 +55,12 @@
     <EmbeddedJar Include="..\..\externals\com.xamarin.google.android.play.feature.delivery.extensions\extensions.jar" />
   </ItemGroup>
 }
+
+@* .NET 9 changes some binding internals that breaks some of our hand bound code. This property reverts
+   to the old behavior. Once we are .NET 9+ we should redo our hand bound code and remove this. *@
+@if (@Model.NuGetPackageId == "Xamarin.AndroidX.Media3.ExoPlayer")
+{
+  <PropertyGroup>
+    <_AndroidEmitLegacyInterfaceInvokers>true</_AndroidEmitLegacyInterfaceInvokers>
+  </PropertyGroup>
+}
diff --git a/source/androidx.paging/paging-common-jvm/Transforms/Metadata.xml b/source/androidx.paging/paging-common-jvm/Transforms/Metadata.xml
index 877384035..6ecbb00d2 100644
--- a/source/androidx.paging/paging-common-jvm/Transforms/Metadata.xml
+++ b/source/androidx.paging/paging-common-jvm/Transforms/Metadata.xml
@@ -109,4 +109,12 @@
         public
     </attr>
 
+    <!-- Workaround generator can't detect this was implemented as an explicitly implemented interface member on a base class -->
+    <attr
+      path="/api/package[@name='androidx.paging']/class[@name='PagedList']"
+      name="skipInvokerMethods"
+      >
+      java/util/SequencedCollection.reversed()Ljava/util/SequencedCollection;
+    </attr>
+
 </metadata>
\ No newline at end of file
diff --git a/source/androidx.work/work-runtime/Transforms/Metadata.xml b/source/androidx.work/work-runtime/Transforms/Metadata.xml
index 5aa1c0894..80c8c0dd4 100644
--- a/source/androidx.work/work-runtime/Transforms/Metadata.xml
+++ b/source/androidx.work/work-runtime/Transforms/Metadata.xml
@@ -11,6 +11,7 @@
     <attr path="/api/package[@name='androidx.work']/class[@name='WorkRequest.Builder']/method[@name='setInputData']" name="managedReturn">AndroidX.Work.WorkRequest.Builder</attr>
     <attr path="/api/package[@name='androidx.work']/class[@name='WorkRequest.Builder']/method[@name='setPeriodStartTime']" name="managedReturn">AndroidX.Work.WorkRequest.Builder</attr>
     <attr path="/api/package[@name='androidx.work']/class[@name='WorkRequest.Builder']/method[@name='setScheduleRequestedAt']" name="managedReturn">AndroidX.Work.WorkRequest.Builder</attr>
+    <attr path="/api/package[@name='androidx.work']/class[@name='WorkRequest.Builder']/method[@name='setId']" name="managedReturn">AndroidX.Work.WorkRequest.Builder</attr>
     
     <remove-node
         path="/api/package[@name='androidx.work']/class[@name='WorkRequest.Builder']/typeParameters" 
diff --git a/source/com.google.protobuf/protobuf-lite/Transforms/Metadata.xml b/source/com.google.protobuf/protobuf-lite/Transforms/Metadata.xml
index 2ebf346ed..64aab0e6f 100644
--- a/source/com.google.protobuf/protobuf-lite/Transforms/Metadata.xml
+++ b/source/com.google.protobuf/protobuf-lite/Transforms/Metadata.xml
@@ -343,4 +343,12 @@
         path="/api/package[@name='com.google.protobuf']/class[@name='LazyStringArrayList']/method[@name='addAll' and count(parameter)=2 and parameter[1][@type='int'] and parameter[2][@type='java.util.Collection&lt;? extends java.lang.String&gt;']]"
         />
 
+    <!-- Workaround generator can't detect this was implemented as an explicitly implemented interface member on a base class -->
+    <attr
+        path="/api/package[@name='com.google.protobuf']/class[@name='AbstractProtobufList']"
+        name="skipInvokerMethods"
+        >
+        java/util/SequencedCollection.reversed()Ljava/util/SequencedCollection;
+    </attr>
+
 </metadata>
diff --git a/source/org.jetbrains.kotlin/kotlin-stdlib/Transforms/Metadata.xml b/source/org.jetbrains.kotlin/kotlin-stdlib/Transforms/Metadata.xml
index d5a31dfb7..76c2db212 100644
--- a/source/org.jetbrains.kotlin/kotlin-stdlib/Transforms/Metadata.xml
+++ b/source/org.jetbrains.kotlin/kotlin-stdlib/Transforms/Metadata.xml
@@ -152,6 +152,7 @@
   <remove-node path="/api/package[@name='kotlin.text']/class[@name='StringsKt___StringsKt']/method[@name='filterIndexed' and count(parameter)=2 and parameter[1][@type='java.lang.String'] and parameter[2][@type='kotlin.jvm.functions.Function2&lt;? super java.lang.Integer, ? super java.lang.Character, java.lang.Boolean&gt;']]" />
   <remove-node path="/api/package[@name='kotlin.text']/class[@name='StringsKt___StringsKt']/method[@name='filterNot' and count(parameter)=2 and parameter[1][@type='java.lang.String'] and parameter[2][@type='kotlin.jvm.functions.Function1&lt;? super java.lang.Character, java.lang.Boolean&gt;']]" />
   <remove-node path="/api/package[@name='kotlin.text']/class[@name='StringsKt___StringsKt']/method[@name='onEach' and count(parameter)=2 and parameter[1][@type='S'] and parameter[2][@type='kotlin.jvm.functions.Function1&lt;? super java.lang.Character, kotlin.Unit&gt;']]/typeParameters" />
+  <attr path="/api/package[@name='kotlin.text']/class[@name='StringsKt___StringsKt']/method[@name='onEach' and count(parameter)=2 and parameter[1][@type='S'] and parameter[2][@type='kotlin.jvm.functions.Function1&lt;? super java.lang.Character, kotlin.Unit&gt;']]" name="return">java.lang.CharSequence</attr>
   <attr path="/api/package[@name='kotlin.text']/class[@name='StringsKt___StringsKt']/method[@name='onEach' and count(parameter)=2 and parameter[1][@type='S'] and parameter[2][@type='kotlin.jvm.functions.Function1&lt;? super java.lang.Character, kotlin.Unit&gt;']]/parameter[1]" name="type">java.lang.CharSequence</attr>
   <remove-node path="/api/package[@name='kotlin.text']/class[@name='StringsKt___StringsKt']/method[@name='slice' and count(parameter)=2 and parameter[1][@type='java.lang.String'] and parameter[2][@type='kotlin.ranges.IntRange']]" />
   <remove-node path="/api/package[@name='kotlin.text']/class[@name='StringsKt___StringsKt']/method[@name='take' and count(parameter)=2 and parameter[1][@type='java.lang.String'] and parameter[2][@type='int']]" />
@@ -444,4 +445,26 @@
 
     <remove-node path="/api/package[@name='kotlin.collections.builders']/class[@name='AbstractMapBuilderEntrySet']/method[@name='contains' and count(parameter)=1 and parameter[1][@type='java.lang.Object']]" />
 
-  </metadata>
+    <!-- Workaround generator can't detect this was implemented as an explicitly implemented interface member on a base class -->
+    <attr
+      path="/api/package[@name='kotlin.collections']/class[@name='AbstractMutableList']"
+      name="skipInvokerMethods"
+      >
+      java/util/SequencedCollection.reversed()Ljava/util/SequencedCollection;
+    </attr>
+    
+    <attr
+      path="/api/package[@name='kotlin.collections']/class[@name='AbstractList']"
+      name="skipInvokerMethods"
+      >
+      java/util/SequencedCollection.reversed()Ljava/util/SequencedCollection;
+    </attr>
+  
+    <!-- Workaround generator doesn't know that `java.util.IList` provides a DIM for this method -->
+    <attr
+      path="/api/package[@name='kotlin.collections']/class[@name='AbstractList']"
+      name="skipInterfaceMethods"
+      >
+      java/util/SequencedCollection.reversed()Ljava/util/SequencedCollection;
+    </attr>
+</metadata>