Move resource busybox actions to BusyBoxActionBuilder and AndroidDataContext

BusyBoxActionBuilder makes much cleaner action builders while making it harder
to do Bad Things (like collapsing NestedSets in analysis, or adding an artifact
to one of the command line and the inputs but not both).

Also:
- In merging, simplify the code somewhat by removing unneeded conditionals -
  for example, the parsed merging action will always be built given the current
  code.
- In the few builders where we aren't doing so already, parameter files should
  always be shell quoted and always be used when the OS is Windows. The BusyBox
  should always support the former and require the latter (although
  sufficiently large inputs may have masked this by triggering parameter files
  in Windows anyway).
- In the builder for linking (within validation), no longer collapse the
  NestedSet of transitiveCompiledSymbols when adding them to the inputs. Using
  the new BusyBoxActionBuilder code, trying to do this would throw an
  IllegalStateException.

RELNOTES: none
PiperOrigin-RevId: 197728382

1 files changed, 2 insertions, 2 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/ParsedAndroidAssets.java b/src/main/java/com/google/devtools/build/lib/rules/android/ParsedAndroidAssets.java
index 265f22f9d6..b1b2971045 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/android/ParsedAndroidAssets.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/android/ParsedAndroidAssets.java
@@ -24,9 +24,9 @@ public class ParsedAndroidAssets extends AndroidAssets implements MergableAndroi
 
   public static ParsedAndroidAssets parseFrom(AndroidDataContext dataContext, AndroidAssets assets)
       throws InterruptedException {
-    return new AndroidResourceParsingActionBuilder(dataContext.getRuleContext())
+    return new AndroidResourceParsingActionBuilder()
         .setOutput(dataContext.createOutputArtifact(AndroidRuleClasses.ANDROID_ASSET_SYMBOLS))
-        .build(assets);
+        .build(dataContext, assets);
   }
 
   public static ParsedAndroidAssets of(AndroidAssets assets, Artifact symbols, Label label) {