diff options
author | tomlu <tomlu@google.com> | 2018-02-09 08:55:52 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-02-09 08:57:25 -0800 |
commit | a4cf463814cc3a0dcffccd4c7038c494395c0998 (patch) | |
tree | 979f15ccb3564a67e14f5bd724669a26b16482c1 /src | |
parent | 8ac761a63cf9cf6851138e689c0666063ad9b219 (diff) |
Make blaze stop passing --rule_kind to java tools.
This argument is unused and should be removed.
RELNOTES: None
PiperOrigin-RevId: 185147327
Diffstat (limited to 'src')
7 files changed, 0 insertions, 39 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java index 2fbbe6db46..aae265a87e 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java +++ b/src/main/java/com/google/devtools/build/lib/rules/android/AndroidCommon.java @@ -591,7 +591,6 @@ public class AndroidCommon { /*disableStrictDeps=*/ useDataBinding); helper.addLibrariesToAttributes(javaCommon.targetsTreatedAsDeps(ClasspathType.COMPILE_ONLY)); - attributes.setRuleKind(ruleContext.getRule().getRuleClass()); attributes.setTargetLabel(ruleContext.getLabel()); JavaCommon.validateConstraint( diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java index b2394ca51d..3628b72380 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCommon.java @@ -639,7 +639,6 @@ public class JavaCommon { addPlugins(javaTargetAttributes); - javaTargetAttributes.setRuleKind(ruleContext.getRule().getRuleClass()); javaTargetAttributes.setTargetLabel(ruleContext.getLabel()); return javaTargetAttributes; diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationHelper.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationHelper.java index 09525dfc28..849b9d0595 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationHelper.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationHelper.java @@ -227,7 +227,6 @@ public final class JavaCompilationHelper { builder.setStrictJavaDeps(attributes.getStrictJavaDeps()); builder.setDirectJars(attributes.getDirectJars()); builder.setCompileTimeDependencyArtifacts(attributes.getCompileTimeDependencyArtifacts()); - builder.setRuleKind(attributes.getRuleKind()); builder.setTargetLabel( attributes.getTargetLabel() == null ? ruleContext.getLabel() : attributes.getTargetLabel()); @@ -406,7 +405,6 @@ public final class JavaCompilationHelper { builder.setStrictJavaDeps(attributes.getStrictJavaDeps()); builder.setCompileTimeDependencyArtifacts(attributes.getCompileTimeDependencyArtifacts()); builder.setDirectJars(attributes.getDirectJars()); - builder.setRuleKind(attributes.getRuleKind()); builder.setTargetLabel(attributes.getTargetLabel()); builder.setAdditionalInputs(NestedSetBuilder.wrap(Order.LINK_ORDER, additionalJavaBaseInputs)); builder.setJavacJar(javaToolchain.getJavac()); diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompileAction.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompileAction.java index 9b2fde252d..42e8543517 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompileAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaCompileAction.java @@ -492,7 +492,6 @@ public final class JavaCompileAction extends SpawnAction { private PathFragment classDirectory; private NestedSet<Artifact> processorPath = NestedSetBuilder.emptySet(Order.NAIVE_LINK_ORDER); private final List<String> processorNames = new ArrayList<>(); - private String ruleKind; private Label targetLabel; /** @@ -714,9 +713,6 @@ public final class JavaCompileAction extends SpawnAction { // terminate --javacopts with `--` to support javac flags that start with `--` result.add("--"); } - if (ruleKind != null) { - result.add("--rule_kind", ruleKind); - } if (targetLabel != null) { result.add("--target_label"); if (targetLabel.getPackageIdentifier().getRepository().isDefault() @@ -1000,11 +996,6 @@ public final class JavaCompileAction extends SpawnAction { return this; } - public Builder setRuleKind(String ruleKind) { - this.ruleKind = ruleKind; - return this; - } - public Builder setTargetLabel(Label targetLabel) { this.targetLabel = targetLabel; return this; diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaHeaderCompileAction.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaHeaderCompileAction.java index 6cffa7b35a..8b8c72a925 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaHeaderCompileAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaHeaderCompileAction.java @@ -189,7 +189,6 @@ public class JavaHeaderCompileAction extends SpawnAction { private NestedSet<Artifact> classpathEntries = NestedSetBuilder.emptySet(Order.NAIVE_LINK_ORDER); private ImmutableList<Artifact> bootclasspathEntries = ImmutableList.<Artifact>of(); - @Nullable private String ruleKind; @Nullable private Label targetLabel; private PathFragment tempDirectory; private BuildConfiguration.StrictDepsMode strictJavaDeps @@ -285,12 +284,6 @@ public class JavaHeaderCompileAction extends SpawnAction { return this; } - /** Sets the kind of the build rule being compiled (e.g. {@code java_library}). */ - public Builder setRuleKind(@Nullable String ruleKind) { - this.ruleKind = ruleKind; - return this; - } - /** Sets the label of the target being compiled. */ public Builder setTargetLabel(@Nullable Label targetLabel) { this.targetLabel = targetLabel; @@ -555,9 +548,6 @@ public class JavaHeaderCompileAction extends SpawnAction { result.add("--"); } - if (ruleKind != null) { - result.add("--rule_kind", ruleKind); - } if (targetLabel != null) { result.add("--target_label"); if (targetLabel.getPackageIdentifier().getRepository().isDefault() diff --git a/src/main/java/com/google/devtools/build/lib/rules/java/JavaTargetAttributes.java b/src/main/java/com/google/devtools/build/lib/rules/java/JavaTargetAttributes.java index 74f9948e18..89a10faad2 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/java/JavaTargetAttributes.java +++ b/src/main/java/com/google/devtools/build/lib/rules/java/JavaTargetAttributes.java @@ -93,7 +93,6 @@ public class JavaTargetAttributes { private final NestedSetBuilder<Artifact> directJars = NestedSetBuilder.naiveLinkOrder(); private final NestedSetBuilder<Artifact> compileTimeDependencyArtifacts = NestedSetBuilder.stableOrder(); - private String ruleKind; private Label targetLabel; private final NestedSetBuilder<Artifact> excludedArtifacts = @@ -176,12 +175,6 @@ public class JavaTargetAttributes { return this; } - public Builder setRuleKind(String ruleKind) { - Preconditions.checkArgument(!built); - this.ruleKind = ruleKind; - return this; - } - public Builder setTargetLabel(Label targetLabel) { Preconditions.checkArgument(!built); this.targetLabel = targetLabel; @@ -373,7 +366,6 @@ public class JavaTargetAttributes { additionalOutputs, directJars.build(), compileTimeDependencyArtifacts.build(), - ruleKind, targetLabel, excludedArtifacts, strictJavaDeps); @@ -444,7 +436,6 @@ public class JavaTargetAttributes { private final NestedSet<Artifact> directJars; private final NestedSet<Artifact> compileTimeDependencyArtifacts; - private final String ruleKind; private final Label targetLabel; private final NestedSet<Artifact> excludedArtifacts; @@ -470,7 +461,6 @@ public class JavaTargetAttributes { Set<Artifact> additionalOutputs, NestedSet<Artifact> directJars, NestedSet<Artifact> compileTimeDependencyArtifacts, - String ruleKind, Label targetLabel, NestedSetBuilder<Artifact> excludedArtifacts, BuildConfiguration.StrictDepsMode strictJavaDeps) { @@ -496,7 +486,6 @@ public class JavaTargetAttributes { this.classPathResources = ImmutableList.copyOf(classPathResources); this.additionalOutputs = ImmutableSet.copyOf(additionalOutputs); this.compileTimeDependencyArtifacts = compileTimeDependencyArtifacts; - this.ruleKind = ruleKind; this.targetLabel = targetLabel; this.excludedArtifacts = excludedArtifacts.build(); this.strictJavaDeps = strictJavaDeps; @@ -631,10 +620,6 @@ public class JavaTargetAttributes { return !messages.isEmpty(); } - public String getRuleKind() { - return ruleKind; - } - public Label getTargetLabel() { return targetLabel; } diff --git a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java index ea752af511..18079f71e3 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java @@ -175,7 +175,6 @@ public class AndroidLibraryTest extends AndroidBuildViewTestCase { (JavaCompileAction) getGeneratingActionForLabel("//java/android:liba.jar"); String commandLine = Iterables.toString(javacAction.buildCommandLine()); - assertThat(commandLine).contains("--rule_kind, android_library"); assertThat(commandLine).contains("--target_label, //java/android:a"); } |