Bug 1285511 - Improve existing build flavor comments. r=sebastian draft
authorMichael Comella <michael.l.comella@gmail.com>
Wed, 20 Jul 2016 15:48:53 -0700
changeset 390262 d9631679e44acea4d31ab547633e0729768f987a
parent 386404 5978434c7789519778ba9f46ec723f06363a7a65
child 390263 106d15f29db183387a77eb3cbc6968a7cac34286
push id23636
push usermichael.l.comella@gmail.com
push dateWed, 20 Jul 2016 22:59:21 +0000
reviewerssebastian
bugs1285511
milestone50.0a1
Bug 1285511 - Improve existing build flavor comments. r=sebastian MozReview-Commit-ID: CRPqN07Pe9V
mobile/android/app/build.gradle
--- a/mobile/android/app/build.gradle
+++ b/mobile/android/app/build.gradle
@@ -40,30 +40,30 @@ android {
         release {
             shrinkResources true
             minifyEnabled true
             proguardFile "${topsrcdir}/mobile/android/config/proguard/proguard.cfg"
         }
     }
 
     productFlavors {
-        // For fast local development.  If you have an API 21 device, you should
-        // use this.
+        // For API 21+ - with multi dex, this will be faster for local development.
         local {
-            // Setting `minSdkVersion 21` allows the Android gradle plugin to
+            // For multi dex, setting `minSdkVersion 21` allows the Android gradle plugin to
             // pre-DEX each module and produce an APK that can be tested on
             // Android Lollipop without time consuming DEX merging processes.
             minSdkVersion 21
             dexOptions {
                 preDexLibraries true
                 multiDexEnabled true
             }
         }
-        // For local development on older devices.  Use this only if you only
-        // have a pre-API 21 device, or if you want to test API-specific things.
+        // For API < 21 - does not support multi dex because local development
+        // is slow in that case. Most builds will not require multi dex so this
+        // should not be an issue.
         localOld {
         }
         // Automation builds.
         automation {
         }
     }
 
     sourceSets {