Revert "[Xamarin.Android.Build.Tasks] Don't package reference assemblies (#706)"

This reverts commit d15-4/0bd26fd3c19fb973316172c10aa2c9e9ae43c1e3.

Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=58679

Turns Out™ that `@(ReferenceCopyLocalPaths)` isn't a "drop-in"
replacement for `@(ReferencePaths)`: `@(ReferenceCopyLocalPaths)`
doesn't contain `Mono.Android.Export.dll`, which means that every app
which uses `Mono.Android.Export.dll` and properly contains a
`<Reference Include="Mono.Android.Export" />` now elicits the
packaging error:

	error XA4210: You need to add a reference to Mono.Android.Export.dll when you use ExportAttribute or ExportFieldAttribute.

Revert the fix for now until we can create a better fix.

Author: jonpryor

Makefile

@@ -148,7 +148,7 @@ TEST_APK_PROJECTS = \
 # Syntax: $(call BUILD_TEST_APK,path/to/project.csproj)
 define BUILD_TEST_APK
 	# Must use xabuild to ensure correct assemblies are resolved
-	MSBUILD="$(MSBUILD)" tools/scripts/xabuild $(MSBUILD_FLAGS) /t:SignAndroidPackage $(1)
+	MSBUILD="$(MSBUILD)" tools/scripts/xabuild /t:SignAndroidPackage $(1)
 endef	# BUILD_TEST_APK
 
 run-apk-tests:

src/Xamarin.Android.Build.Tasks/Tasks/BuildApk.cs

@@ -275,10 +275,6 @@ private void AddAssemblies (ZipArchiveEx apk)
 
 			int count = 0;
 			foreach (ITaskItem assembly in ResolvedUserAssemblies) {
-
-				if (MonoAndroidHelper.IsReferenceAssembly (assembly.ItemSpec)) {
-					Log.LogWarning ($"{assembly.ItemSpec} is a reference assembly!");
-				}
 				// Add assembly
 				apk.Archive.AddFile (assembly.ItemSpec, GetTargetDirectory (assembly.ItemSpec) + "/"  + Path.GetFileName (assembly.ItemSpec), compressionMethod: CompressionMethod.Store);
 
@@ -311,9 +307,6 @@ private void AddAssemblies (ZipArchiveEx apk)
 			count = 0;
 			// Add framework assemblies
 			foreach (ITaskItem assembly in ResolvedFrameworkAssemblies) {
-				if (MonoAndroidHelper.IsReferenceAssembly (assembly.ItemSpec)) {
-					Log.LogWarning ($"{assembly.ItemSpec} is a reference assembly!");
-				}
 				apk.Archive.AddFile (assembly.ItemSpec, "assemblies/" + Path.GetFileName (assembly.ItemSpec), compressionMethod: CompressionMethod.Store);
 				var config = Path.ChangeExtension (assembly.ItemSpec, "dll.config");
 				AddAssemblyConfigEntry (apk, config);

src/Xamarin.Android.Build.Tasks/Utilities/MonoAndroidHelper.cs

@@ -8,7 +8,6 @@
 using Mono.Security.Cryptography;
 using Xamarin.Android.Build.Utilities;
 using Xamarin.Tools.Zip;
-using Mono.Cecil;
 
 #if MSBUILD
 using Microsoft.Build.Framework;
@@ -261,14 +260,6 @@ public static bool IsFrameworkAssembly (string assembly, bool checkSdkPath)
 			return TargetFrameworkDirectories == null || !checkSdkPath ? false : ExistsInFrameworkPath (assembly);
 		}
 
-		public static bool IsReferenceAssembly (string assembly)
-		{
-			var a = AssemblyDefinition.ReadAssembly (assembly, new ReaderParameters() { InMemory = true, ReadSymbols = false, });
-			if (!a.HasCustomAttributes)
-				return false;
-			return a.CustomAttributes.Any (t => t.AttributeType.FullName == "System.Runtime.CompilerServices.ReferenceAssemblyAttribute");
-		}
-
 		public static bool ExistsInFrameworkPath (string assembly)
 		{
 			return TargetFrameworkDirectories

src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets

@@ -1429,11 +1429,7 @@ because xbuild doesn't support framework reference assemblies.
 <Target Name="_ResolveAssemblies">
 	<!--- Remove the ImplicitlyExpandDesignTimeFacades assemblies. We have already build the app there are not required for packaging  -->
 	<ItemGroup>
-		<FilteredAssemblies Include="%(ReferenceCopyLocalPaths.Identity)"
-				Condition="'%(ReferenceCopyLocalPaths.ResolvedFrom)' != 'ImplicitlyExpandDesignTimeFacades' And '%(ReferenceCopyLocalPaths.Extension)' == '.dll' And '%(ReferenceCopyLocalPaths.DestinationSubDirectory)' == '' "/>
-		<!-- Fallback to @(ReferencePath) if @(ReferenceCopyLocalPaths) is empty. This is for xbuild support -->
-		<FilteredAssemblies Include="%(ReferencePath.Identity)"
-				Condition="'%(ReferencePath.ResolvedFrom)' != 'ImplicitlyExpandDesignTimeFacades' And '@(ReferenceCopyLocalPaths)' == '' "/>
+		<FilteredAssemblies Include="@(ReferencePath)" Condition="'%(ReferencePath.ResolvedFrom)' != 'ImplicitlyExpandDesignTimeFacades'" />
 	</ItemGroup>
 	<!-- Find all the assemblies this app requires -->
 	<ResolveAssemblies