diff options
author | 2017-09-07 16:56:52 +0200 | |
---|---|---|
committer | 2017-09-08 09:47:21 +0200 | |
commit | d6a0c413b17e54215ce17707184ba7c7295014a4 (patch) | |
tree | ae6c2ef8bc2d4ab13cf91005b0d5c695d887ba1b /src/main/java/com/google | |
parent | e4262e3bddde8eb78a80a2ce952c1556bc967cce (diff) |
Make --experimental_android_use_nocompress_extensions_on_apk a no-op.
RELNOTES: android_binary.nocompress_extensions now applies to all files in the
APK, not just resources and assets.
PiperOrigin-RevId: 167859686
Diffstat (limited to 'src/main/java/com/google')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java | 12 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/android/ApkActionsBuilder.java | 3 |
2 files changed, 3 insertions, 12 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java index ccc5b4c0f1..9b95e95f15 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java +++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java @@ -666,10 +666,8 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { name = "experimental_android_use_nocompress_extensions_on_apk", defaultValue = "false", documentationCategory = OptionDocumentationCategory.UNDOCUMENTED, - effectTags = {OptionEffectTag.UNKNOWN}, - help = - "Use the value of nocompress_extensions attribute with the SingleJar " - + "--nocompress_suffixes flag when building the APK." + effectTags = {OptionEffectTag.NO_OP}, + help = "This flag is deprecated and has no effect. It will be removed in a future release." ) public boolean useNocompressExtensionsOnApk; @@ -807,7 +805,6 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { private final ResourceFilter resourceFilter; private final boolean compressJavaResources; private final boolean includeLibraryResourceJars; - private final boolean useNocompressExtensionsOnApk; private final boolean exportsManifestDefault; private final AndroidAaptVersion androidAaptVersion; private final boolean generateRobolectricRClass; @@ -847,7 +844,6 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { this.resourceFilter = options.resourceFilter; this.compressJavaResources = options.compressJavaResources; this.includeLibraryResourceJars = options.includeLibraryResourceJars; - this.useNocompressExtensionsOnApk = options.useNocompressExtensionsOnApk; this.exportsManifestDefault = options.exportsManifestDefault; this.androidAaptVersion = options.androidAaptVersion; this.generateRobolectricRClass = options.generateRobolectricRClass; @@ -974,10 +970,6 @@ public class AndroidConfiguration extends BuildConfiguration.Fragment { return includeLibraryResourceJars; } - boolean useNocompressExtensionsOnApk() { - return useNocompressExtensionsOnApk; - } - boolean getExportsManifestDefault() { return exportsManifestDefault; } diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/ApkActionsBuilder.java b/src/main/java/com/google/devtools/build/lib/rules/android/ApkActionsBuilder.java index 7c1b8aa171..bd1f60e25a 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/android/ApkActionsBuilder.java +++ b/src/main/java/com/google/devtools/build/lib/rules/android/ApkActionsBuilder.java @@ -333,8 +333,7 @@ public class ApkActionsBuilder { ImmutableList<String> noCompressExtensions = ruleContext.getTokenizedStringListAttr("nocompress_extensions"); - if (ruleContext.getFragment(AndroidConfiguration.class).useNocompressExtensionsOnApk() - && !noCompressExtensions.isEmpty()) { + if (!noCompressExtensions.isEmpty()) { compressedApkCommandLine.addAll("--nocompress_suffixes", noCompressExtensions); singleJarCommandLine.addAll("--nocompress_suffixes", noCompressExtensions); } |