diff options
author | jcater <jcater@google.com> | 2018-05-01 13:20:00 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-05-01 13:22:33 -0700 |
commit | 36745914cc043df70e8c50c975ef125593406c8a (patch) | |
tree | e12c97ec70c489942b180dd2d5055cbca8f85819 /src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java | |
parent | 26ff4b3e3997aab79e39caf62c0d123a315d9478 (diff) |
Clean up code that directly imports nested classes like Builder, Entry, etc.
PiperOrigin-RevId: 194985259
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java b/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java index ca48a59e4b..712be737a2 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java @@ -38,7 +38,6 @@ import com.google.devtools.build.lib.packages.Attribute.LabelLateBoundDefault; import com.google.devtools.build.lib.packages.Attribute.LabelListLateBoundDefault; import com.google.devtools.build.lib.packages.AttributeMap; import com.google.devtools.build.lib.packages.RuleClass; -import com.google.devtools.build.lib.packages.RuleClass.Builder; import com.google.devtools.build.lib.packages.RuleClass.Builder.RuleClassType; import com.google.devtools.build.lib.packages.TestSize; import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec; @@ -99,7 +98,7 @@ public class BaseRuleClasses { */ public static final class TestBaseRule implements RuleDefinition { @Override - public RuleClass build(Builder builder, RuleDefinitionEnvironment env) { + public RuleClass build(RuleClass.Builder builder, RuleDefinitionEnvironment env) { return builder .requiresConfigurationFragments(TestConfiguration.class) .add( @@ -255,7 +254,7 @@ public class BaseRuleClasses { public static final class RootRule implements RuleDefinition { @Override - public RuleClass build(Builder builder, RuleDefinitionEnvironment environment) { + public RuleClass build(RuleClass.Builder builder, RuleDefinitionEnvironment environment) { return nameAttribute(builder).build(); } @@ -311,7 +310,7 @@ public class BaseRuleClasses { */ public static final class MakeVariableExpandingRule implements RuleDefinition { @Override - public RuleClass build(Builder builder, RuleDefinitionEnvironment env) { + public RuleClass build(RuleClass.Builder builder, RuleDefinitionEnvironment env) { return builder /* <!-- #BLAZE_RULE($make_variable_expanding_rule).ATTRIBUTE(toolchains) --> The set of toolchains that supply <a href="${link make-variables}">"Make variables"</a> @@ -338,7 +337,7 @@ public class BaseRuleClasses { */ public static final class RuleBase implements RuleDefinition { @Override - public RuleClass build(Builder builder, RuleDefinitionEnvironment env) { + public RuleClass build(RuleClass.Builder builder, RuleDefinitionEnvironment env) { return builder .add(attr("deps", LABEL_LIST).legacyAllowAnyFileType()) .add(attr("data", LABEL_LIST).cfg(env.getLipoDataTransition()) @@ -363,7 +362,7 @@ public class BaseRuleClasses { /** A base rule for all binary rules. */ public static final class BinaryBaseRule implements RuleDefinition { @Override - public RuleClass build(Builder builder, RuleDefinitionEnvironment env) { + public RuleClass build(RuleClass.Builder builder, RuleDefinitionEnvironment env) { return builder .add(attr("args", STRING_LIST)) .add(attr("output_licenses", LICENSE)) @@ -387,7 +386,7 @@ public class BaseRuleClasses { /** Rule class for rules in error. */ public static final class ErrorRule implements RuleDefinition { @Override - public RuleClass build(Builder builder, RuleDefinitionEnvironment env) { + public RuleClass build(RuleClass.Builder builder, RuleDefinitionEnvironment env) { return builder.publicByDefault().build(); } |