Distinguish mnemonics between different usages of aapt.

RELNOTES: None PiperOrigin-RevId: 162945309
bazelbuild · Jul 24, 2017 · e619d6a · e619d6a
1 parent ad42824
commit e619d6a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidAaptActionHelper.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidAaptActionHelper.java
@@ -111,7 +111,7 @@ public void createGenerateResourceSymbolsAction(Artifact javaSourcesJar,
             .setCommandLine(CommandLine.of(args))
             .useParameterFile(ParameterFileType.UNQUOTED)
             .setProgressMessage("Generating Java resources")
-            .setMnemonic("AndroidAapt");
+            .setMnemonic("AaptJavaGenerator");
     if (rTxt != null) {
       builder.addOutput(rTxt);
     }
@@ -152,7 +152,7 @@ public void createGenerateApkAction(Artifact apk, String renameManifestPackage,
             .setCommandLine(CommandLine.of(args))
             .useParameterFile(ParameterFileType.UNQUOTED)
             .setProgressMessage("Generating apk resources")
-            .setMnemonic("AndroidAapt")
+            .setMnemonic("AaptResourceApk")
             .build(ruleContext));
   }
 
@@ -218,7 +218,7 @@ public List<String> getArgs(Artifact output, String actionKind, ResourceType res
     if (args.size() > 3) {
       return ImmutableList.copyOf(args);
     } else {
-      return ImmutableList.<String>of();
+      return ImmutableList.of();
     }
   }
 
@@ -280,7 +280,7 @@ public void createGenerateProguardAction(
             .setCommandLine(CommandLine.of(aaptCommand))
             .useParameterFile(ParameterFileType.UNQUOTED)
             .setProgressMessage("Generating Proguard configuration for resources")
-            .setMnemonic("AndroidAapt")
+            .setMnemonic("AaptProguardConfiguration")
             .build(ruleContext));
   }
 }
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidBinary.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidBinary.java
@@ -1732,7 +1732,7 @@ private static Artifact createSplitApkResources(RuleContext ruleContext,
     AndroidSdkProvider sdk = AndroidSdkProvider.fromRuleContext(ruleContext);
     ruleContext.registerAction(new SpawnAction.Builder()
         .setExecutable(sdk.getAapt())
-        .setMnemonic("AndroidAapt")
+        .setMnemonic("AaptSplitResourceApk")
         .setProgressMessage("Generating resource apk for split " + splitName)
         .addArgument("package")
         .addArgument("-F")

diff --git a/...in/java/com/google/devtools/build/lib/rules/android/AndroidResourcesProcessorBuilder.java b/...in/java/com/google/devtools/build/lib/rules/android/AndroidResourcesProcessorBuilder.java
@@ -337,12 +337,12 @@ private ResourceContainer createAaptAction(ActionConstructionContext context) {
             .useParameterFile(ParameterFileType.UNQUOTED)
             .addTool(sdk.getAapt())
             .addTransitiveInputs(inputs.build())
-            .addOutputs(ImmutableList.<Artifact>copyOf(outs))
+            .addOutputs(ImmutableList.copyOf(outs))
             .setCommandLine(builder.build())
             .setExecutable(
                 ruleContext.getExecutablePrerequisite("$android_resources_busybox", Mode.HOST))
             .setProgressMessage("Processing Android resources for " + ruleContext.getLabel())
-            .setMnemonic("AndroidAapt")
+            .setMnemonic("AaptPackage")
             .build(context));
 
     // Return the full set of processed transitive dependencies.