diff options
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/genrule')
3 files changed, 1 insertions, 21 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/genrule/BUILD b/src/main/java/com/google/devtools/build/lib/rules/genrule/BUILD index 81ab46717a..5c97ae1988 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/genrule/BUILD +++ b/src/main/java/com/google/devtools/build/lib/rules/genrule/BUILD @@ -12,7 +12,6 @@ java_library( "//src/main/java/com/google/devtools/build/lib:build-base", "//src/main/java/com/google/devtools/build/lib:events", "//src/main/java/com/google/devtools/build/lib:java-compilation", - "//src/main/java/com/google/devtools/build/lib:java-implicit-attributes", "//src/main/java/com/google/devtools/build/lib:packages", "//src/main/java/com/google/devtools/build/lib:vfs", "//src/main/java/com/google/devtools/build/lib/actions", diff --git a/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBase.java b/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBase.java index 1aab9b1648..407a45cb3d 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBase.java +++ b/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBase.java @@ -64,7 +64,7 @@ public abstract class GenRuleBase implements RuleConfiguredTargetFactory { return CROSSTOOL_MAKE_VARIABLE.matcher(command).find(); } - protected static boolean requiresJdk(String command) { + protected boolean requiresJdk(String command) { return JDK_MAKE_VARIABLE.matcher(command).find(); } diff --git a/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBaseRule.java b/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBaseRule.java index d4cb75844f..544a03df21 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBaseRule.java +++ b/src/main/java/com/google/devtools/build/lib/rules/genrule/GenRuleBaseRule.java @@ -37,9 +37,6 @@ import com.google.devtools.build.lib.packages.RuleClass.Builder.RuleClassType; import com.google.devtools.build.lib.rules.MakeVariableProvider; import com.google.devtools.build.lib.rules.cpp.CppConfiguration; import com.google.devtools.build.lib.rules.cpp.CppRuleClasses; -import com.google.devtools.build.lib.rules.java.JavaImplicitAttributes; -import com.google.devtools.build.lib.rules.java.JavaSemantics; -import com.google.devtools.build.lib.rules.java.Jvm; import com.google.devtools.build.lib.syntax.Type; import com.google.devtools.build.lib.util.FileTypeSet; @@ -66,22 +63,6 @@ public class GenRuleBaseRule implements RuleDefinition { } }; - /** - * Late-bound dependency on the host JDK <i>iff</i> the genrule has make variables that need - * that rule. - */ - public static final Attribute.LateBoundLabel<BuildConfiguration> HOST_JDK = - new Attribute.LateBoundLabel<BuildConfiguration>( - JavaImplicitAttributes.JDK_LABEL, Jvm.class) { - @Override - public Label resolve(Rule rule, AttributeMap attributes, BuildConfiguration configuration) { - return attributes != null - && GenRuleBase.requiresJdk(attributes.get("cmd", Type.STRING)) - ? JavaSemantics.HOST_JDK.resolve(rule, attributes, configuration) - : null; - } - }; - @Override public RuleClass build( RuleClass.Builder builder, RuleDefinitionEnvironment env) { |