[Xamarin.Android.Build.Tasks] Emit XA1038 for deprecated properties (#…

…8511)

Context: 43243b4
Context: 8928f11

Previously we have added the following temporary "fallback flags" for
bindings-related changes:

  - `$(_AndroidUseJavaLegacyResolver)`
  - `$(_AndroidEmitLegacyInterfaceInvokers)`

Because the intent is to remove the old, deprecated code, we need to
warn users of these flags that they will be removed in a future
version of .NET.

Also fix a typo that prevented
`$(_AndroidEmitLegacyInterfaceInvokers)` from being able to be set.

I don't know that it's worth having unit tests for this;
locally this produces:

	…\bin\Debug\lib\packs\Microsoft.Android.Sdk.Windows\34.99.0\tools\Xamarin.Android.Bindings.Core.targets(37,5): warning XA1037: The '_AndroidUseJavaLegacyResolver' MSBuild property is deprecated and will be removed in .NET 10. See https://aka.ms/net-android-deprecations for more details.
	…\bin\Debug\lib\packs\Microsoft.Android.Sdk.Windows\34.99.0\tools\Xamarin.Android.Bindings.Core.targets(42,5): warning XA1037: The '_AndroidEmitLegacyInterfaceInvokers' MSBuild property is deprecated and will be removed in .NET 10. See https://aka.ms/net-android-deprecations for more details.

Documentation/guides/messages/README.md

@@ -138,6 +138,7 @@ or 'Help->Report a Problem' in Visual Studio for Mac.
 + [XA1035](xa1035.md): The 'BundleAssemblies' property is deprecated and it has no effect on the application build. Equivalent functionality is implemented by the 'AndroidUseAssemblyStore' and 'AndroidEnableAssemblyCompression' properties.
 + [XA1036](xa1036.md): AndroidManifest.xml //uses-sdk/@android:minSdkVersion '29' does not match the $(SupportedOSPlatformVersion) value '21' in the project file (if there is no $(SupportedOSPlatformVersion) value in the project file, then a default value has been assumed).
 Either change the value in the AndroidManifest.xml to match the $(SupportedOSPlatformVersion) value, or remove the value in the AndroidManifest.xml (and add a $(SupportedOSPlatformVersion) value to the project file if it doesn't already exist).
++ [XA1037](xa1037.md): The '{0}' MSBuild property is deprecated and will be removed in .NET {1}. See https://aka.ms/net-android-deprecations for more details.
 
 ## XA2xxx: Linker
 

Documentation/guides/messages/xa1037.md

@@ -0,0 +1,22 @@
+---
+title: Xamarin.Android warning XA1037
+description: XA1037 warning code
+ms.date: 27/11/2023
+---
+# Xamarin.Android warning XA1037
+
+## Example messages
+
+```
+The '_AndroidUseJavaLegacyResolver' MSBuild property is deprecated and will be removed in .NET 10. 
+See https://aka.ms/net-android-deprecations for more details.
+```
+
+## Solution
+
+Edit your csproj directly and remove the referenced MSBuild property.
+
+Test your project to ensure the new behavior is functionally equivalent.
+
+If not, file an [issue](https://github.com/xamarin/xamarin-android/issues) so a 
+solution can be found before the deprecated flag is removed.

src/Xamarin.Android.Build.Tasks/MSBuild/Xamarin/Android/Xamarin.Android.Bindings.Core.targets

@@ -33,6 +33,19 @@ It is shared between "legacy" binding projects and .NET 5 projects.
     <NoWarn Condition=" '$(DocumentationFile)' != '' ">$(NoWarn);CS1573;CS1591</NoWarn>
   </PropertyGroup>
 
+  <Target Name="_CheckNonIdealBindingConfigurations">
+    <AndroidWarning Code="XA1037"
+        ResourceName="XA1037"
+        FormatArguments="_AndroidUseJavaLegacyResolver;10"
+        Condition=" '$(_AndroidUseJavaLegacyResolver)' == 'true' "
+    />
+    <AndroidWarning Code="XA1037"
+        ResourceName="XA1037"
+        FormatArguments="_AndroidEmitLegacyInterfaceInvokers;10"
+        Condition=" '$(_AndroidEmitLegacyInterfaceInvokers)' == 'true' "
+    />
+  </Target>
+
   <Target Name="_SetAndroidGenerateManagedBindings"
       Condition=" '@(InputJar->Count())' != '0' Or '@(EmbeddedJar->Count())' != '0' Or '@(LibraryProjectZip->Count())' != '0' Or '@(_JavaBindingSource->Count())' != '0' ">
     <PropertyGroup>

src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.BuildOrder.targets

@@ -82,6 +82,7 @@ properties that determine build ordering.
   <PropertyGroup Condition=" '$(AndroidApplication)' != 'True' ">
     <BuildDependsOn>
       _ValidateLinkMode;
+      _CheckNonIdealBindingConfigurations;
       _SetupMSBuildAllProjects;
       _SetupDesignTimeBuildForBuild;
       _CategorizeAndroidLibraries;

src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.DefaultProperties.targets

@@ -39,7 +39,7 @@
     <AndroidBoundInterfacesContainStaticAndDefaultInterfaceMethods Condition=" '$(AndroidBoundInterfacesContainStaticAndDefaultInterfaceMethods)' == '' ">true</AndroidBoundInterfacesContainStaticAndDefaultInterfaceMethods>
     <AndroidBoundInterfacesContainTypes Condition=" '$(AndroidBoundInterfacesContainTypes)' == '' ">true</AndroidBoundInterfacesContainTypes>
     <AndroidBoundInterfacesContainConstants Condition=" '$(AndroidBoundInterfacesContainConstants)' == '' ">true</AndroidBoundInterfacesContainConstants>
-    <_AndroidEmitLegacyInterfaceInvokers Condition=" '$(AndroidEmitLegacyInterfaceInvokers)' == '' ">false</_AndroidEmitLegacyInterfaceInvokers>
+    <_AndroidEmitLegacyInterfaceInvokers Condition=" '$(_AndroidEmitLegacyInterfaceInvokers)' == '' ">false</_AndroidEmitLegacyInterfaceInvokers>
     <AndroidEnableObsoleteOverrideInheritance Condition=" '$(AndroidEnableObsoleteOverrideInheritance)' == '' ">true</AndroidEnableObsoleteOverrideInheritance>
     <AndroidEnableRestrictToAttributes Condition=" '$(AndroidEnableRestrictToAttributes)' == '' ">obsolete</AndroidEnableRestrictToAttributes>
 

src/Xamarin.Android.Build.Tasks/Properties/Resources.resx

@@ -996,4 +996,10 @@ To use a custom JDK path for a command line build, set the 'JavaSdkDirectory' MS
     <comment>The following are literal names and should not be translated: Maven
 {0} - User supplied Maven repository type</comment>
   </data>
+  <data name="XA1037" xml:space="preserve">
+    <value>The '{0}' MSBuild property is deprecated and will be removed in .NET {1}. See https://aka.ms/net-android-deprecations for more details.</value>
+    <comment>The following are literal names and should not be translated: MSBuild, .NET.
+{0} - The deprecated MSBuild property name
+{1} - The numeric version of .NET</comment>
+  </data>
 </root>