diff --git a/.gitmodules b/.gitmodules
index 17af4eba7c0..48f228fa79b 100644
--- a/.gitmodules
+++ b/.gitmodules
@@ -41,6 +41,9 @@
path = external/xamarin-android-tools
url = https://github.com/xamarin/xamarin-android-tools
branch = master
+[submodule "external/r8"]
+ path = external/r8
+ url = https://r8.googlesource.com/r8
[submodule "external/dlfcn-win32"]
path = external/dlfcn-win32
url = https://github.com/dlfcn-win32/dlfcn-win32.git
diff --git a/Documentation/guides/BuildProcess.md b/Documentation/guides/BuildProcess.md
index deca390dcd6..215b0a018be 100644
--- a/Documentation/guides/BuildProcess.md
+++ b/Documentation/guides/BuildProcess.md
@@ -210,6 +210,15 @@ when packaing Release applications.
This property is `False` by default.
+- **AndroidEnableDesugar** – A boolean property that
+ determines if `desugar` is enabled. Android does not currently
+ support all Java 8 features, and the default toolchain implements
+ the new language features by performing bytecode transformations,
+ called `desugar`, on the output of the `javac` compiler.
+
+- **AndroidEnableD8** – A boolean property that determines
+ whether or not use Google's new dexer (dx.jar alternative).
+
- **AndroidEnableMultiDex** – A boolean property that
determines whether or not multi-dex support will be used in the
final `.apk`.
@@ -419,6 +428,10 @@ when packaing Release applications.
This property should be `True` for Debug builds, and `False` for
Release projects.
+- **AndroidUseR8** – A boolean property that determines
+ whether or not use Google's new Java code shrinker (proguard
+ alternative).
+
- **AotAssemblies** – A boolean property that determines
whether or not assemblies will be Ahead-of-Time compiled into
native code and included in the `.apk`.
diff --git a/Documentation/guides/messages/xa4304.md b/Documentation/guides/messages/xa4304.md
index 384c2b38478..f554ea1c2b4 100644
--- a/Documentation/guides/messages/xa4304.md
+++ b/Documentation/guides/messages/xa4304.md
@@ -1,4 +1,4 @@
-# Compiler Error XA4304
+# Compiler Warning XA4304
The `Proguard` MSBuild task encountered a proguard configuration file
that was not found on disk. These files are generally declared in your
diff --git a/Documentation/guides/messages/xa4305.md b/Documentation/guides/messages/xa4305.md
new file mode 100644
index 00000000000..bb6036ba6fb
--- /dev/null
+++ b/Documentation/guides/messages/xa4305.md
@@ -0,0 +1,21 @@
+# Compiler Error/Warning XA4305
+
+The `CreateMultiDexMainDexClassList`, `CompileToDalvik` or `R8`
+MSBuild task encountered a `multidex.keep` file that was not found on
+disk. You can customize `multidex` settings for your Xamarin.Android
+application by adding files with the `MultiDexMainDexList` build item,
+which are combined into a final `multidex.keep` file.
+
+To learn more about `multidex` and how it relates to Android
+development, see the [Android documentation][android].
+
+## Resolution
+
+Verify you are not declaring a `MultiDexMainDexList` build item that
+does not exist.
+
+Consider submitting a [bug][bug] if you are getting this error/warning
+under normal circumstances.
+
+[android]: https://developer.android.com/studio/build/multidex
+[bug]: https://github.com/xamarin/xamarin-android/wiki/Submitting-Bugs,-Feature-Requests,-and-Pull-Requests
diff --git a/Xamarin.Android.sln b/Xamarin.Android.sln
index b53520fd5ea..89e51cdc8ce 100644
--- a/Xamarin.Android.sln
+++ b/Xamarin.Android.sln
@@ -123,6 +123,8 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Xamarin.Android.Tools.Andro
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Xamarin.Android.Tools.AndroidSdk-Tests", "external\xamarin-android-tools\src\Xamarin.Android.Tools.AndroidSdk\Tests\Xamarin.Android.Tools.AndroidSdk-Tests.csproj", "{1E5501E8-49C1-4659-838D-CC9720C5208F}"
EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "r8", "src\r8\r8.csproj", "{1BAFA0CC-0377-46CE-AB7B-7BB2E7B62F63}"
+EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "proprietary", "build-tools\proprietary\proprietary.csproj", "{D93CAC27-3893-42A3-99F1-2BCA72E186F4}"
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "download-bundle", "build-tools\download-bundle\download-bundle.csproj", "{1DA0CB12-5508-4E83-A242-0C8D6D99A49B}"
@@ -358,6 +360,10 @@ Global
{1DA0CB12-5508-4E83-A242-0C8D6D99A49B}.Debug|AnyCPU.Build.0 = Debug|Any CPU
{1DA0CB12-5508-4E83-A242-0C8D6D99A49B}.Release|AnyCPU.ActiveCfg = Release|Any CPU
{1DA0CB12-5508-4E83-A242-0C8D6D99A49B}.Release|AnyCPU.Build.0 = Release|Any CPU
+ {1BAFA0CC-0377-46CE-AB7B-7BB2E7B62F63}.Debug|AnyCPU.ActiveCfg = Debug|Any CPU
+ {1BAFA0CC-0377-46CE-AB7B-7BB2E7B62F63}.Debug|AnyCPU.Build.0 = Debug|Any CPU
+ {1BAFA0CC-0377-46CE-AB7B-7BB2E7B62F63}.Release|AnyCPU.ActiveCfg = Release|Any CPU
+ {1BAFA0CC-0377-46CE-AB7B-7BB2E7B62F63}.Release|AnyCPU.Build.0 = Release|Any CPU
EndGlobalSection
GlobalSection(SolutionProperties) = preSolution
HideSolutionNode = FALSE
@@ -418,8 +424,7 @@ Global
{B8105878-D423-4159-A3E7-028298281EC6} = {04E3E11E-B47D-4599-8AFC-50515A95E715}
{E34BCFA0-CAA4-412C-AA1C-75DB8D67D157} = {04E3E11E-B47D-4599-8AFC-50515A95E715}
{1E5501E8-49C1-4659-838D-CC9720C5208F} = {CAB438D8-B0F5-4AF0-BEBD-9E2ADBD7B483}
- {D93CAC27-3893-42A3-99F1-2BCA72E186F4} = {E351F97D-EA4F-4E7F-AAA0-8EBB1F2A4A62}
- {1DA0CB12-5508-4E83-A242-0C8D6D99A49B} = {E351F97D-EA4F-4E7F-AAA0-8EBB1F2A4A62}
+ {1BAFA0CC-0377-46CE-AB7B-7BB2E7B62F63} = {04E3E11E-B47D-4599-8AFC-50515A95E715}
EndGlobalSection
GlobalSection(ExtensibilityGlobals) = postSolution
SolutionGuid = {53A1F287-EFB2-4D97-A4BB-4A5E145613F6}
diff --git a/external/r8 b/external/r8
new file mode 160000
index 00000000000..671655efb15
--- /dev/null
+++ b/external/r8
@@ -0,0 +1 @@
+Subproject commit 671655efb1567746e11b7f89c3c1c2a3c6283f8b
diff --git a/external/r8.tpnitems b/external/r8.tpnitems
new file mode 100644
index 00000000000..8d945808d81
--- /dev/null
+++ b/external/r8.tpnitems
@@ -0,0 +1,9 @@
+
+
+
+
+ $(MSBuildThisFileDirectory)\r8\LICENSE
+ https://r8.googlesource.com/r8/
+
+
+
diff --git a/src/Mono.Android/Test/Mono.Android-Tests.csproj b/src/Mono.Android/Test/Mono.Android-Tests.csproj
index 001a4bf5194..5e9a14584a4 100644
--- a/src/Mono.Android/Test/Mono.Android-Tests.csproj
+++ b/src/Mono.Android/Test/Mono.Android-Tests.csproj
@@ -20,6 +20,8 @@
False
v8.1
true
+ True
+ True
diff --git a/src/Xamarin.Android.Build.Tasks/Tasks/CompileToDalvik.cs b/src/Xamarin.Android.Build.Tasks/Tasks/CompileToDalvik.cs
index c516a5187bc..c862b2df774 100644
--- a/src/Xamarin.Android.Build.Tasks/Tasks/CompileToDalvik.cs
+++ b/src/Xamarin.Android.Build.Tasks/Tasks/CompileToDalvik.cs
@@ -116,8 +116,14 @@ protected override string GenerateCommandLineCommands ()
cmd.AppendSwitchIfNotNull ("--input-list=", inputListFile);
if (MultiDexEnabled) {
- cmd.AppendSwitch ("--multi-dex");
- cmd.AppendSwitchIfNotNull ("--main-dex-list=", MultiDexMainDexListFile);
+ if (string.IsNullOrEmpty (MultiDexMainDexListFile)) {
+ Log.LogCodedWarning ("XA4305", $"MultiDex is enabled, but '{nameof (MultiDexMainDexListFile)}' was not specified.");
+ } else if (!File.Exists (MultiDexMainDexListFile)) {
+ Log.LogCodedWarning ("XA4305", MultiDexMainDexListFile, 0, $"MultiDex is enabled, but main dex list file '{MultiDexMainDexListFile}' does not exist.");
+ } else {
+ cmd.AppendSwitch ("--multi-dex");
+ cmd.AppendSwitchIfNotNull ("--main-dex-list=", MultiDexMainDexListFile);
+ }
}
cmd.AppendSwitchIfNotNull ("--output ", Path.GetDirectoryName (ClassesOutputDirectory));
diff --git a/src/Xamarin.Android.Build.Tasks/Tasks/CreateMultiDexMainDexClassList.cs b/src/Xamarin.Android.Build.Tasks/Tasks/CreateMultiDexMainDexClassList.cs
index a138f28a91c..5fd5fe9e400 100644
--- a/src/Xamarin.Android.Build.Tasks/Tasks/CreateMultiDexMainDexClassList.cs
+++ b/src/Xamarin.Android.Build.Tasks/Tasks/CreateMultiDexMainDexClassList.cs
@@ -62,9 +62,17 @@ public override bool Execute ()
var result = base.Execute () && !Log.HasLoggedErrors;
- if (result && CustomMainDexListFiles != null && CustomMainDexListFiles.Any (x => File.Exists (x.ItemSpec))) {
- foreach (var content in CustomMainDexListFiles.Select (i => File.ReadAllLines (i.ItemSpec)))
- File.AppendAllLines (MultiDexMainDexListFile, content);
+ if (result && CustomMainDexListFiles?.Length > 0) {
+ var content = new List ();
+ foreach (var file in CustomMainDexListFiles) {
+ if (File.Exists (file.ItemSpec)) {
+ content.Add (File.ReadAllText (file.ItemSpec));
+ } else {
+ Log.LogCodedError ("XA4305", file.ItemSpec, 0, $"'MultiDexMainDexList' file '{file.ItemSpec}' does not exist.");
+ }
+ }
+ File.WriteAllText (MultiDexMainDexListFile, string.Concat (content));
+ return true;
}
return result;
diff --git a/src/Xamarin.Android.Build.Tasks/Tasks/Proguard.cs b/src/Xamarin.Android.Build.Tasks/Tasks/Proguard.cs
index 3ed19e8afc4..744b7f683e6 100644
--- a/src/Xamarin.Android.Build.Tasks/Tasks/Proguard.cs
+++ b/src/Xamarin.Android.Build.Tasks/Tasks/Proguard.cs
@@ -41,23 +41,17 @@ public class Proguard : ToolTask
[Required]
public string ProguardJarOutput { get; set; }
- [Required]
public string ProguardGeneratedReferenceConfiguration { get; set; }
-
- [Required]
public string ProguardGeneratedApplicationConfiguration { get; set; }
-
- [Required]
public string ProguardCommonXamarinConfiguration { get; set; }
+ [Required]
public string ProguardConfigurationFiles { get; set; }
public ITaskItem[] JavaLibrariesToEmbed { get; set; }
- public ITaskItem[] ExternalJavaLibraries { get; set; }
+ public ITaskItem[] JavaLibrariesToReference { get; set; }
- public ITaskItem[] DoNotPackageJavaLibraries { get; set; }
-
public bool UseProguard { get; set; }
public string JavaOptions { get; set; }
@@ -119,15 +113,9 @@ protected override string GenerateCommandLineCommands ()
// skip invalid lines
}
- var injars = new List ();
- var libjars = new List ();
- injars.Add (classesZip);
- if (JavaLibrariesToEmbed != null)
- foreach (var jarfile in JavaLibrariesToEmbed)
- injars.Add (jarfile.ItemSpec);
-
- using (var xamcfg = File.Create (ProguardCommonXamarinConfiguration))
- GetType ().Assembly.GetManifestResourceStream ("proguard_xamarin.cfg").CopyTo (xamcfg);
+ if (!string.IsNullOrWhiteSpace (ProguardCommonXamarinConfiguration))
+ using (var xamcfg = File.Create (ProguardCommonXamarinConfiguration))
+ GetType ().Assembly.GetManifestResourceStream ("proguard_xamarin.cfg").CopyTo (xamcfg);
var configs = ProguardConfigurationFiles
.Replace ("{sdk.dir}", AndroidSdkDirectory + Path.DirectorySeparatorChar)
@@ -148,9 +136,15 @@ protected override string GenerateCommandLineCommands ()
Log.LogCodedWarning ("XA4304", file, 0, "Proguard configuration file '{0}' was not found.", file);
}
+ var injars = new List ();
+ var libjars = new List ();
+ injars.Add (classesZip);
+ if (JavaLibrariesToEmbed != null)
+ foreach (var jarfile in JavaLibrariesToEmbed)
+ injars.Add (jarfile.ItemSpec);
libjars.Add (JavaPlatformJarPath);
- if (ExternalJavaLibraries != null)
- foreach (var jarfile in ExternalJavaLibraries.Select (p => p.ItemSpec))
+ if (JavaLibrariesToReference != null)
+ foreach (var jarfile in JavaLibrariesToReference.Select (p => p.ItemSpec))
libjars.Add (jarfile);
cmd.AppendSwitchUnquotedIfNotNull ("-injars ", "\"'" + string.Join ($"'{ProguardInputJarFilter}{Path.PathSeparator}'", injars.Distinct ()) + $"'{ProguardInputJarFilter}\"");
diff --git a/src/Xamarin.Android.Build.Tasks/Tasks/R8.cs b/src/Xamarin.Android.Build.Tasks/Tasks/R8.cs
new file mode 100644
index 00000000000..1fbe6054ca0
--- /dev/null
+++ b/src/Xamarin.Android.Build.Tasks/Tasks/R8.cs
@@ -0,0 +1,147 @@
+// Copyright (C) 2018 Xamarin, Inc. All rights reserved.
+
+using System;
+using System.Linq;
+using System.IO;
+using Microsoft.Build.Framework;
+using Microsoft.Build.Utilities;
+using System.Collections.Generic;
+using Xamarin.Android.Tools;
+
+namespace Xamarin.Android.Tasks
+{
+
+ public class R8 : JavaToolTask
+ {
+ [Required]
+ public string R8JarPath { get; set; }
+
+ [Required]
+ public string OutputDirectory { get; set; }
+
+ public string Configuration { get; set; }
+
+ // It is loaded to calculate --min-api, which is used by desugaring part to determine which levels of desugaring it performs.
+ [Required]
+ public string AndroidManifestFile { get; set; }
+
+ // general r8 feature options.
+ public bool EnableDesugar { get; set; }
+ public bool EnableMinify { get; set; } // The Task has the option, but it is not supported at all.
+ public bool EnableTreeShaking { get; set; }
+
+ // Java libraries to embed or reference
+ [Required]
+ public string ClassesZip { get; set; }
+ [Required]
+ public string JavaPlatformJarPath { get; set; }
+ public ITaskItem [] JavaLibrariesToEmbed { get; set; }
+ public ITaskItem [] JavaLibrariesToReference { get; set; }
+
+ // used for proguard configuration settings
+ [Required]
+ public string AndroidSdkDirectory { get; set; }
+ [Required]
+ public string AcwMapFile { get; set; }
+ public string ProguardGeneratedReferenceConfiguration { get; set; }
+ public string ProguardGeneratedApplicationConfiguration { get; set; }
+ public string ProguardCommonXamarinConfiguration { get; set; }
+ [Required]
+ public string ProguardConfigurationFiles { get; set; }
+ public string ProguardMappingOutput { get; set; }
+
+ // multidex
+ public bool EnableMultiDex { get; set; }
+ public string MultiDexMainDexListFile { get; set; }
+
+ public string R8ExtraArguments { get; set; }
+
+ protected override string GenerateCommandLineCommands ()
+ {
+ var cmd = new CommandLineBuilder ();
+
+ cmd.AppendSwitchIfNotNull ("-jar ", R8JarPath);
+
+ if (!string.IsNullOrEmpty (R8ExtraArguments))
+ cmd.AppendSwitch (R8ExtraArguments); // it should contain "--dex".
+ if (Configuration.Equals ("Debug", StringComparison.OrdinalIgnoreCase))
+ cmd.AppendSwitch ("--debug");
+
+ // generating proguard application configuration
+ if (EnableTreeShaking) {
+ var acwLines = File.ReadAllLines (AcwMapFile);
+ using (var appcfg = File.CreateText (ProguardGeneratedApplicationConfiguration))
+ for (int i = 0; i + 2 < acwLines.Length; i += 3)
+ try {
+ var line = acwLines [i + 2];
+ var java = line.Substring (line.IndexOf (';') + 1);
+ appcfg.WriteLine ("-keep class " + java + " { *; }");
+ } catch {
+ // skip invalid lines
+ }
+ if (!string.IsNullOrWhiteSpace (ProguardCommonXamarinConfiguration))
+ using (var xamcfg = File.Create (ProguardCommonXamarinConfiguration))
+ GetType ().Assembly.GetManifestResourceStream ("proguard_xamarin.cfg").CopyTo (xamcfg);
+ var configs = ProguardConfigurationFiles
+ .Replace ("{sdk.dir}", AndroidSdkDirectory + Path.DirectorySeparatorChar)
+ .Replace ("{intermediate.common.xamarin}", ProguardCommonXamarinConfiguration)
+ .Replace ("{intermediate.references}", ProguardGeneratedReferenceConfiguration)
+ .Replace ("{intermediate.application}", ProguardGeneratedApplicationConfiguration)
+ .Replace ("{project}", string.Empty) // current directory anyways.
+ .Split (';')
+ .Select (s => s.Trim ())
+ .Where (s => !string.IsNullOrWhiteSpace (s));
+ foreach (var file in configs) {
+ if (File.Exists (file))
+ cmd.AppendSwitchIfNotNull ("--pg-conf ", file);
+ else
+ Log.LogCodedWarning ("XA4304", file, 0, "Proguard configuration file '{0}' was not found.", file);
+ }
+ cmd.AppendSwitchIfNotNull ("--pg-map-output ", ProguardMappingOutput);
+ }
+
+ // multidexing
+ if (EnableMultiDex) {
+ if (string.IsNullOrEmpty (MultiDexMainDexListFile)) {
+ Log.LogCodedWarning ("XA4305", $"MultiDex is enabled, but '{nameof (MultiDexMainDexListFile)}' was not specified.");
+ } else if (!File.Exists (MultiDexMainDexListFile)) {
+ Log.LogCodedWarning ("XA4305", MultiDexMainDexListFile, 0, $"MultiDex is enabled, but main dex list file '{MultiDexMainDexListFile}' does not exist.");
+ } else {
+ cmd.AppendSwitchIfNotNull ("--main-dex-list ", MultiDexMainDexListFile);
+ }
+ }
+
+ // desugaring
+ var doc = AndroidAppManifest.Load (AndroidManifestFile, MonoAndroidHelper.SupportedVersions);
+ int minApiVersion = doc.MinSdkVersion == null ? 4 : (int)doc.MinSdkVersion;
+ cmd.AppendSwitchIfNotNull ("--min-api ", minApiVersion.ToString ());
+
+ if (!EnableTreeShaking)
+ cmd.AppendSwitch ("--no-tree-shaking");
+ if (!EnableDesugar)
+ cmd.AppendSwitch ("--no-desugaring");
+ if (!EnableMinify)
+ cmd.AppendSwitch ("--no-minification");
+
+ var injars = new List ();
+ var libjars = new List ();
+ injars.Add (ClassesZip);
+ if (JavaLibrariesToEmbed != null)
+ foreach (var jarfile in JavaLibrariesToEmbed)
+ injars.Add (jarfile.ItemSpec);
+ libjars.Add (JavaPlatformJarPath);
+ if (JavaLibrariesToReference != null)
+ foreach (var jarfile in JavaLibrariesToReference.Select (p => p.ItemSpec))
+ libjars.Add (jarfile);
+
+ cmd.AppendSwitchIfNotNull ("--output ", OutputDirectory);
+ foreach (var jar in libjars)
+ cmd.AppendSwitchIfNotNull ("--lib ", jar);
+ foreach (var jar in injars)
+ cmd.AppendFileNameIfNotNull (jar);
+
+ return cmd.ToString ();
+ }
+ }
+
+}
diff --git a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.OSS.cs b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.OSS.cs
index 6bce79f2d34..ebcda802f78 100644
--- a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.OSS.cs
+++ b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.OSS.cs
@@ -60,21 +60,49 @@ public partial class BuildTest : BaseTest
/* isRelease */ true,
/* enableProguard */ true,
/* useLatestSdk */ true,
+ /* useR8 */ false,
},
new Object [] {
/* isRelease */ true,
/* enableProguard */ false,
/* useLatestSdk */ true,
+ /* useR8 */ false,
},
new Object [] {
/* isRelease */ false,
/* enableProguard */ true,
/* useLatestSdk */ true,
+ /* useR8 */ false,
},
new Object [] {
/* isRelease */ false,
/* enableProguard */ false,
/* useLatestSdk */ true,
+ /* useR8 */ false,
+ },
+ new Object [] {
+ /* isRelease */ true,
+ /* enableProguard */ true,
+ /* useLatestSdk */ true,
+ /* useR8 */ true,
+ },
+ new Object [] {
+ /* isRelease */ true,
+ /* enableProguard */ false,
+ /* useLatestSdk */ true,
+ /* useR8 */ true,
+ },
+ new Object [] {
+ /* isRelease */ false,
+ /* enableProguard */ true,
+ /* useLatestSdk */ true,
+ /* useR8 */ true,
+ },
+ new Object [] {
+ /* isRelease */ false,
+ /* enableProguard */ false,
+ /* useLatestSdk */ true,
+ /* useR8 */ true,
},
};
static object [] TakeSimpleFlag = new object [] {
diff --git a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.cs b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.cs
index b50aaf48064..d0d39287703 100644
--- a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.cs
+++ b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.cs
@@ -662,17 +662,22 @@ public void BuildAotApplicationAndBundle (string supportedAbis, bool enableLLVM,
[Test]
[TestCaseSource ("ProguardChecks")]
- public void BuildProguardEnabledProject (bool isRelease, bool enableProguard, bool useLatestSdk)
+ public void BuildProguardEnabledProject (bool isRelease, bool enableProguard, bool useLatestSdk, bool useR8)
{
- var proj = new XamarinAndroidApplicationProject () { IsRelease = isRelease, EnableProguard = enableProguard, UseLatestPlatformSdk = useLatestSdk, TargetFrameworkVersion = useLatestSdk ? "v7.1" : "v5.0" };
- using (var b = CreateApkBuilder (Path.Combine ("temp", $"BuildProguard Enabled Project(1){isRelease}{enableProguard}{useLatestSdk}"))) {
+ var proj = new XamarinAndroidApplicationProject () { IsRelease = isRelease, EnableProguard = enableProguard, UseR8 = useR8, UseLatestPlatformSdk = useLatestSdk, TargetFrameworkVersion = useLatestSdk ? "v7.1" : "v5.0" };
+ using (var b = CreateApkBuilder (Path.Combine ("temp", $"BuildProguard Enabled Project(1){isRelease}{enableProguard}{useLatestSdk}{useR8}"))) {
Assert.IsTrue (b.Build (proj), "Build should have succeeded.");
if (isRelease && enableProguard) {
var proguardProjectPrimary = Path.Combine (Root, b.ProjectDirectory, proj.IntermediateOutputPath, "proguard", "proguard_project_primary.cfg");
FileAssert.Exists (proguardProjectPrimary);
- StringAssertEx.ContainsText (File.ReadAllLines (proguardProjectPrimary), "-keep class md52d9cf6333b8e95e8683a477bc589eda5.MainActivity");
+ Assert.IsTrue (StringAssertEx.ContainsText (File.ReadAllLines (proguardProjectPrimary), "-keep class md52d9cf6333b8e95e8683a477bc589eda5.MainActivity"), "`md52d9cf6333b8e95e8683a477bc589eda5.MainActivity` should exist in `proguard_project_primary.cfg`!");
}
+
+ var className = "Lmono/MonoRuntimeProvider;";
+ var dexFile = b.Output.GetIntermediaryPath (Path.Combine ("android", "bin", "classes.dex"));
+ FileAssert.Exists (dexFile);
+ Assert.IsTrue (DexUtils.ContainsClass (className, dexFile, b.AndroidSdkDirectory), $"`{dexFile}` should include `{className}`!");
}
}
@@ -2155,12 +2160,13 @@ public void BuildReleaseApplication ()
}
[Test]
- public void BuildApplicationWithSpacesInPath ([Values (true, false)] bool isRelease, [Values (true, false)] bool enableProguard, [Values (true, false)] bool enableMultiDex)
+ public void BuildApplicationWithSpacesInPath ([Values (true, false)] bool isRelease, [Values (true, false)] bool enableProguard, [Values (true, false)] bool enableMultiDex, [Values (true, false)] bool useR8)
{
var proj = new XamarinAndroidApplicationProject () {
IsRelease = isRelease,
AotAssemblies = isRelease,
EnableProguard = enableProguard,
+ UseR8 = useR8,
};
proj.OtherBuildItems.Add (new BuildItem ("AndroidJavaLibrary", "Hello (World).jar") { BinaryContent = () => Convert.FromBase64String (@"
UEsDBBQACAgIAMl8lUsAAAAAAAAAAAAAAAAJAAQATUVUQS1JTkYv/soAAAMAUEsHCAAAAAACAAAAA
@@ -2192,9 +2198,14 @@ public void BuildApplicationWithSpacesInPath ([Values (true, false)] bool isRele
",
});
}
- using (var b = CreateApkBuilder (Path.Combine ("temp", $"BuildReleaseAppWithA InIt({isRelease}{enableProguard}{enableMultiDex})"))) {
+ using (var b = CreateApkBuilder (Path.Combine ("temp", $"BuildReleaseAppWithA InIt({isRelease}{enableProguard}{enableMultiDex}{useR8})"))) {
Assert.IsTrue (b.Build (proj), "Build should have succeeded.");
Assert.IsFalse (b.LastBuildOutput.ContainsText ("Duplicate zip entry"), "Should not get warning about [META-INF/MANIFEST.MF]");
+
+ var className = "Lmono/MonoRuntimeProvider;";
+ var dexFile = b.Output.GetIntermediaryPath (Path.Combine ("android", "bin", "classes.dex"));
+ FileAssert.Exists (dexFile);
+ Assert.IsTrue (DexUtils.ContainsClass (className, dexFile, b.AndroidSdkDirectory), $"`{dexFile}` should include `{className}`!");
}
}
@@ -3140,6 +3151,7 @@ public void foo()
if (enableDesugar) {
var className = "Lmono/MonoRuntimeProvider;";
var dexFile = builder.Output.GetIntermediaryPath (Path.Combine ("android", "bin", "classes.dex"));
+ FileAssert.Exists (dexFile);
Assert.IsTrue (DexUtils.ContainsClass (className, dexFile, builder.AndroidSdkDirectory), $"`{dexFile}` should include `{className}`!");
}
}
diff --git a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/KnownProperties.cs b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/KnownProperties.cs
index c0d0ca37714..b6c3de96664 100644
--- a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/KnownProperties.cs
+++ b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/KnownProperties.cs
@@ -17,6 +17,7 @@ public static class KnownProperties
public const string BundleAssemblies = "BundleAssemblies";
public const string EnableProguard = "EnableProguard";
public const string AndroidEnableDesugar = "AndroidEnableDesugar";
+ public const string AndroidUseR8 = "AndroidUseR8";
public const string UseJackAndJill = "UseJackAndJill";
public const string AotAssemblies = "AotAssemblies";
diff --git a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/XamarinAndroidApplicationProject.cs b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/XamarinAndroidApplicationProject.cs
index a3804b50f52..ee4643e518a 100644
--- a/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/XamarinAndroidApplicationProject.cs
+++ b/src/Xamarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/XamarinAndroidApplicationProject.cs
@@ -85,6 +85,11 @@ public bool EnableDesugar {
set { SetProperty (KnownProperties.AndroidEnableDesugar, value.ToString ()); }
}
+ public bool UseR8 {
+ get { return string.Equals (GetProperty (KnownProperties.AndroidUseR8), "True", StringComparison.OrdinalIgnoreCase); }
+ set { SetProperty (KnownProperties.AndroidUseR8, value.ToString ()); }
+ }
+
public string AndroidFastDeploymentType {
get { return GetProperty (KnownProperties.AndroidFastDeploymentType); }
set { SetProperty (KnownProperties.AndroidFastDeploymentType, value); }
diff --git a/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Build.Tasks.csproj b/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Build.Tasks.csproj
index 989eed5a633..52253c9fd8f 100644
--- a/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Build.Tasks.csproj
+++ b/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Build.Tasks.csproj
@@ -569,6 +569,7 @@
+
@@ -732,6 +733,11 @@
proguard
False
+
+ {1bafa0cc-0377-46ce-ab7b-7bb2e7b62f63}
+ r8
+ False
+
{E248B2CA-303B-4645-ADDC-9D4459D550FD}
libZipSharp
diff --git a/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets b/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets
index e2de6b13c22..2ba5bedaf67 100755
--- a/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets
+++ b/src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets
@@ -73,6 +73,7 @@ Copyright (C) 2011-2012 Xamarin. All rights reserved.
+
@@ -280,6 +281,7 @@ Copyright (C) 2011-2012 Xamarin. All rights reserved.
$(EnableProguard)
False
False
+ False
1G
@@ -936,12 +938,36 @@ because xbuild doesn't support framework reference assemblies.
/>
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+