commit 4e71d54ff210b36e4cd1554406f759eb08a87cb9
author cushon <cushon@google.com> Fri Nov 13 16:04:59 2020 -0800
committer Copybara-Service <copybara-worker@google.com> Fri Nov 13 16:05:56 2020 -0800
tree f0f216c97bfcd97bb476da12a00e29b8a0553d37
parent 3edc5affa1f3bf3053821e0aa762395cb9475376

Allow proguard whitelister to be a deploy jar

PiperOrigin-RevId: 342357126

src/main/java/com/google/devtools/build/lib/rules/java/ProguardLibrary.java

diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/ProguardLibrary.java b/src/main/java/com/google/devtools/build/lib/rules/java/ProguardLibrary.java
index 25893c7..ff0f34c 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/java/ProguardLibrary.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/java/ProguardLibrary.java
@@ -73,7 +73,7 @@
         return specsBuilder.build();
       }
       for (Artifact specToValidate : localSpecs) {
-        specsBuilder.add(validateProguardSpec(proguardAllowlister, specToValidate));
+        specsBuilder.add(validateProguardSpec(ruleContext, proguardAllowlister, specToValidate));
       }
     }
 
@@ -109,7 +109,7 @@
    * validated Proguard spec, ready to be exported.
    */
   private Artifact validateProguardSpec(
-      FilesToRunProvider proguardAllowlister, Artifact specToValidate) {
+      RuleContext ruleContext, FilesToRunProvider proguardAllowlister, Artifact specToValidate) {
     // If we're validating j/a/b/testapp/proguard.cfg, the output will be:
     // j/a/b/testapp/proguard.cfg_valid
     Artifact output =
@@ -119,19 +119,29 @@
                 .getRootRelativePath()
                 .replaceName(specToValidate.getFilename() + "_valid"),
             ruleContext.getBinOrGenfilesDirectory());
-    ruleContext.registerAction(
-        new SpawnAction.Builder()
-            .addInput(specToValidate)
-            .addOutput(output)
-            .setExecutable(proguardAllowlister)
-            .setProgressMessage("Validating proguard configuration")
-            .setMnemonic("ValidateProguard")
-            .addCommandLine(
-                CustomCommandLine.builder()
-                    .addExecPath("--path", specToValidate)
-                    .addExecPath("--output", output)
-                    .build())
-            .build(ruleContext));
+    SpawnAction.Builder builder =
+        new SpawnAction.Builder().addInput(specToValidate).addOutput(output);
+    if (proguardAllowlister.getExecutable().getExtension().equals("jar")) {
+      builder
+          .setJarExecutable(
+              JavaCommon.getHostJavaExecutable(ruleContext),
+              proguardAllowlister.getExecutable(),
+              JavaToolchainProvider.from(ruleContext).getJvmOptions())
+          .addTransitiveInputs(JavaRuntimeInfo.forHost(ruleContext).javaBaseInputsMiddleman());
+    } else {
+      // TODO(b/170769708): remove this branch and require java_toolchain.proguard_allowlister to
+      // always be a _deploy.jar
+      builder.setExecutable(proguardAllowlister);
+    }
+    builder
+        .setProgressMessage("Validating proguard configuration")
+        .setMnemonic("ValidateProguard")
+        .addCommandLine(
+            CustomCommandLine.builder()
+                .addExecPath("--path", specToValidate)
+                .addExecPath("--output", output)
+                .build());
+    ruleContext.registerAction(builder.build(ruleContext));
     return output;
   }
 }