diff options
author | tomlu <tomlu@google.com> | 2017-08-10 19:26:34 +0200 |
---|---|---|
committer | Marcel Hlopko <hlopko@google.com> | 2017-08-11 12:56:11 +0200 |
commit | 4435515d156dbb0cc40869de686d326b175f61b8 (patch) | |
tree | 4e15de1bd9693857c867d6ef11bf4aee52714eaf /src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java | |
parent | bc4eb273de3ac59962d6e78628a28a9109e43715 (diff) |
Inline @Deprecated methods in CustomCommandLine.
Apart from updating CustomCommandLineTest this CL is entirely automated.
We also sneak in a rename of addFormat -> addFormatted.
RELNOTES: None
PiperOrigin-RevId: 164870140
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java b/src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java index 3039d68b36..89386d6d0c 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java +++ b/src/main/java/com/google/devtools/build/lib/rules/android/RClassGeneratorActionBuilder.java @@ -23,6 +23,7 @@ import com.google.devtools.build.lib.actions.ParameterFile.ParameterFileType; import com.google.devtools.build.lib.analysis.RuleConfiguredTarget.Mode; import com.google.devtools.build.lib.analysis.RuleContext; import com.google.devtools.build.lib.analysis.actions.CustomCommandLine; +import com.google.devtools.build.lib.analysis.actions.CustomCommandLine.VectorArg; import com.google.devtools.build.lib.analysis.actions.SpawnAction; import com.google.devtools.build.lib.collect.nestedset.NestedSet; import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder; @@ -88,11 +89,11 @@ public class RClassGeneratorActionBuilder { List<Artifact> outs = new ArrayList<>(); if (primary.getRTxt() != null) { - builder.addExecPath("--primaryRTxt", primary.getRTxt()); + builder.add("--primaryRTxt", primary.getRTxt()); inputs.add(primary.getRTxt()); } if (primary.getManifest() != null) { - builder.addExecPath("--primaryManifest", primary.getManifest()); + builder.add("--primaryManifest", primary.getManifest()); inputs.add(primary.getManifest()); } if (!Strings.isNullOrEmpty(primary.getJavaPackage())) { @@ -102,9 +103,11 @@ public class RClassGeneratorActionBuilder { // TODO(corysmith): Remove NestedSet as we are already flattening it. Iterable<ResourceContainer> depResources = dependencies.getResources(); if (!Iterables.isEmpty(depResources)) { - builder.addBeforeEach( - "--library", - ImmutableList.copyOf(Iterables.transform(depResources, chooseDepsToArg(version)))); + builder.add( + VectorArg.of( + ImmutableList.copyOf( + Iterables.transform(depResources, chooseDepsToArg(version)))) + .beforeEach("--library")); inputs.addTransitive( NestedSetBuilder.wrap( Order.NAIVE_LINK_ORDER, @@ -112,7 +115,7 @@ public class RClassGeneratorActionBuilder { .transformAndConcat(chooseDepsToArtifacts(version)))); } } - builder.addExecPath("--classJarOutput", classJarOut); + builder.add("--classJarOutput", classJarOut); outs.add(classJarOut); // Create the spawn action. |