Stop AndroidResourceProcessingAction from creating a "_resources" folder in the source tree when invoked from ApplicationManifest and AndroidLibrary.

--
MOS_MIGRATED_REVID=103648946

3 files changed, 0 insertions, 14 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java
index 1c69f385c3..bbbe3df7a9 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidLibrary.java
@@ -170,7 +170,6 @@ public abstract class AndroidLibrary implements RuleConfiguredTargetFactory {
               .withDependencies(transitiveResources)
               .setDebug(
                   ruleContext.getConfiguration().getCompilationMode() != CompilationMode.OPT)
-              .setWorkingDirectory(ruleContext.getUniqueDirectory("_resources"))
               .build(ruleContext);
     }
 
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidResourcesProcessorBuilder.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidResourcesProcessorBuilder.java
index e313a877a4..1da4f5b658 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidResourcesProcessorBuilder.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidResourcesProcessorBuilder.java
@@ -27,7 +27,6 @@ import com.google.devtools.build.lib.analysis.actions.ActionConstructionContext;
 import com.google.devtools.build.lib.analysis.actions.SpawnAction;
 import com.google.devtools.build.lib.rules.android.AndroidResourcesProvider.ResourceContainer;
 import com.google.devtools.build.lib.rules.android.AndroidResourcesProvider.ResourceType;
-import com.google.devtools.build.lib.vfs.PathFragment;
 
 import java.util.ArrayList;
 import java.util.Collections;
@@ -50,7 +49,6 @@ public class AndroidResourcesProcessorBuilder {
   private final AndroidSdkProvider sdk;
   private List<String> assetsToIgnore = Collections.emptyList();
   private SpawnAction.Builder spawnActionBuilder;
-  private PathFragment workingDirectory;
   private List<String> densities = Collections.emptyList();
   private String customJavaPackage;
   private final RuleContext ruleContext;
@@ -133,11 +131,6 @@ public class AndroidResourcesProcessorBuilder {
     return this;
   }
 
-  public AndroidResourcesProcessorBuilder setWorkingDirectory(PathFragment workingDirectory) {
-    this.workingDirectory = workingDirectory;
-    return this;
-  }
-
   private void addResourceContainer(List<Artifact> inputs, List<String> args,
       ResourceContainer container) {
     Iterables.addAll(inputs, container.getArtifacts());
@@ -204,11 +197,6 @@ public class AndroidResourcesProcessorBuilder {
       args.add(Joiner.on(",").join(data));
     }
 
-    if (rTxtOut != null || sourceJarOut != null) {
-      args.add("--generatedSourcePath");
-      args.add(workingDirectory.toString());
-    }
-
     if (rTxtOut != null) {
       args.add("--rOutput");
       args.add(rTxtOut.getExecPathString());
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/ApplicationManifest.java b/src/main/java/com/google/devtools/build/lib/rules/android/ApplicationManifest.java
index 7fade53dc8..7be804a50c 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/ApplicationManifest.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/ApplicationManifest.java
@@ -324,7 +324,6 @@ public final class ApplicationManifest {
             .setDebug(ruleContext.getConfiguration().getCompilationMode() != CompilationMode.OPT)
             .withPrimary(resourceContainer)
             .withDependencies(resourceContainers)
-            .setWorkingDirectory(ruleContext.getUniqueDirectory("_resources"))
             .setDensities(densities)
             .setProguardOut(proguardCfg)
             .setApplicationId(applicationId)