From 488b7b08cb1bb78c0bb0f3952cefdad25846e417 Mon Sep 17 00:00:00 2001 From: Dean Ellis Date: Wed, 4 May 2016 11:14:13 +0100 Subject: [PATCH] [Xamarin.Android.Build.Tasks] Fix the inclusion of .config files in the Apk. Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=40804 There was an issue with including config files in the final apk when using release builds. Firstly they were not even being picked up because the CopyConfigFiles task was expecting a list of config files not a list of assemblies which is what it was receiving. Secondly the BuildApk task was not placing them in the assemblies directory. This commit fixes both of those issues. --- src/Xamarin.Android.Build.Tasks/Tasks/BuildApk.cs | 2 +- .../Xamarin.Android.Common.targets | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Xamarin.Android.Build.Tasks/Tasks/BuildApk.cs b/src/Xamarin.Android.Build.Tasks/Tasks/BuildApk.cs index 7a3c9349a86..624ccaaf102 100644 --- a/src/Xamarin.Android.Build.Tasks/Tasks/BuildApk.cs +++ b/src/Xamarin.Android.Build.Tasks/Tasks/BuildApk.cs @@ -278,7 +278,7 @@ private void AddAssemblies (ZipFile apk) apk.AddFile (assembly.ItemSpec, "assemblies").CompressionLevel = CompressionLevel.None; var config = Path.ChangeExtension (assembly.ItemSpec, "dll.config"); if (File.Exists (config)) - apk.AddFile (config).CompressionLevel = CompressionLevel.None; + apk.AddFile (config, "assemblies").CompressionLevel = CompressionLevel.None; // Try to add symbols if Debug if (debug) { var symbols = Path.ChangeExtension (assembly.ItemSpec, "dll.mdb"); diff --git a/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets b/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets index 2e75b389d9e..ad45f671b20 100755 --- a/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets +++ b/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets @@ -2075,8 +2075,8 @@ because xbuild doesn't support framework reference assemblies. SkipUnchangedFiles="true" /> + SourceFiles="@(_ResolvedFrameworkAssemblies->'%(Identity).config')" + DestinationFiles="@(_ShrunkFrameworkAssemblies->'%(Identity).config')" />