diff options
author | ulfjack <ulfjack@google.com> | 2017-08-02 20:30:07 +0200 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2017-08-03 12:09:54 +0200 |
commit | 1e509a3c82b31f120763b121380d15e01090ab0a (patch) | |
tree | a612803f65146a487fccae6d11fefb428eff2751 /src/main/java/com/google | |
parent | 202119ad33627a8a64ad686c159e6cbee7edcec1 (diff) |
Move core rules to a top-level class.
PiperOrigin-RevId: 164013246
Diffstat (limited to 'src/main/java/com/google')
3 files changed, 87 insertions, 43 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/BUILD b/src/main/java/com/google/devtools/build/lib/BUILD index c1aacb5e35..fb1870b6e8 100644 --- a/src/main/java/com/google/devtools/build/lib/BUILD +++ b/src/main/java/com/google/devtools/build/lib/BUILD @@ -692,6 +692,7 @@ java_library( ":buildeventstream", ":collect", ":concurrent", + ":core-rules", ":events", ":java-compilation", ":java-rules", @@ -1051,6 +1052,19 @@ java_library( ) java_library( + name = "core-rules", + srcs = glob( + ["rules/core/*.java"], + ), + deps = [ + ":build-base", + "//src/main/java/com/google/devtools/build/lib/analysis/featurecontrol", + "//src/main/java/com/google/devtools/build/lib/rules/config", + "//third_party:guava", + ], +) + +java_library( name = "android-rules", srcs = glob( ["rules/android/*.java"], diff --git a/src/main/java/com/google/devtools/build/lib/bazel/rules/BazelRuleClassProvider.java b/src/main/java/com/google/devtools/build/lib/bazel/rules/BazelRuleClassProvider.java index e93a154d44..87de50cd52 100644 --- a/src/main/java/com/google/devtools/build/lib/bazel/rules/BazelRuleClassProvider.java +++ b/src/main/java/com/google/devtools/build/lib/bazel/rules/BazelRuleClassProvider.java @@ -19,7 +19,6 @@ import static com.google.common.base.Preconditions.checkNotNull; import com.google.common.base.Functions; import com.google.common.collect.ImmutableList; import com.google.common.collect.ImmutableSet; -import com.google.devtools.build.lib.analysis.BaseRuleClasses; import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider; import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider.Builder; import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider.RuleSet; @@ -27,8 +26,6 @@ import com.google.devtools.build.lib.analysis.PlatformConfigurationLoader; import com.google.devtools.build.lib.analysis.PlatformOptions; import com.google.devtools.build.lib.analysis.config.BuildConfiguration; import com.google.devtools.build.lib.analysis.constraints.EnvironmentRule; -import com.google.devtools.build.lib.analysis.featurecontrol.FeaturePolicyLoader; -import com.google.devtools.build.lib.analysis.featurecontrol.FeaturePolicyOptions; import com.google.devtools.build.lib.bazel.rules.BazelToolchainType.BazelToolchainTypeRule; import com.google.devtools.build.lib.bazel.rules.android.AndroidNdkRepositoryRule; import com.google.devtools.build.lib.bazel.rules.android.AndroidSdkRepositoryRule; @@ -100,6 +97,7 @@ import com.google.devtools.build.lib.rules.config.ConfigFeatureFlag; import com.google.devtools.build.lib.rules.config.ConfigFeatureFlagConfiguration; import com.google.devtools.build.lib.rules.config.ConfigRuleClasses; import com.google.devtools.build.lib.rules.config.ConfigSkylarkCommon; +import com.google.devtools.build.lib.rules.core.CoreRules; import com.google.devtools.build.lib.rules.cpp.CcIncLibraryRule; import com.google.devtools.build.lib.rules.cpp.CcToolchainAlias; import com.google.devtools.build.lib.rules.cpp.CcToolchainRule; @@ -232,29 +230,6 @@ public class BazelRuleClassProvider { public static final ImmutableSet<String> FEATURE_POLICY_FEATURES = ImmutableSet.of(ConfigFeatureFlag.POLICY_NAME); - public static final RuleSet CORE_RULES = - new RuleSet() { - @Override - public void init(Builder builder) { - builder.addConfigurationOptions(FeaturePolicyOptions.class); - builder.addConfigurationFragment(new FeaturePolicyLoader(FEATURE_POLICY_FEATURES)); - builder.addDynamicTransitionMaps(BaseRuleClasses.DYNAMIC_TRANSITIONS_MAP); - - builder.addRuleDefinition(new BaseRuleClasses.RootRule()); - builder.addRuleDefinition(new BaseRuleClasses.BaseRule()); - builder.addRuleDefinition(new BaseRuleClasses.RuleBase()); - builder.addRuleDefinition(new BaseRuleClasses.MakeVariableExpandingRule()); - builder.addRuleDefinition(new BaseRuleClasses.BinaryBaseRule()); - builder.addRuleDefinition(new BaseRuleClasses.TestBaseRule()); - builder.addRuleDefinition(new BaseRuleClasses.ErrorRule()); - } - - @Override - public ImmutableList<RuleSet> requires() { - return ImmutableList.of(); - } - }; - public static final RuleSet PLATFORM_RULES = new RuleSet() { @Override @@ -273,7 +248,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -298,7 +273,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -318,7 +293,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -333,7 +308,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -349,7 +324,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -362,7 +337,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -378,7 +353,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -410,7 +385,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES); + return ImmutableList.of(CoreRules.INSTANCE); } }; @@ -425,7 +400,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, CPP_RULES); + return ImmutableList.of(CoreRules.INSTANCE, CPP_RULES); } }; @@ -469,7 +444,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, CPP_RULES); + return ImmutableList.of(CoreRules.INSTANCE, CPP_RULES); } }; @@ -490,7 +465,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, JAVA_RULES); + return ImmutableList.of(CoreRules.INSTANCE, JAVA_RULES); } }; @@ -537,7 +512,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, CPP_RULES, JAVA_RULES); + return ImmutableList.of(CoreRules.INSTANCE, CPP_RULES, JAVA_RULES); } }; @@ -559,7 +534,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, CPP_RULES); + return ImmutableList.of(CoreRules.INSTANCE, CPP_RULES); } }; @@ -627,7 +602,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, CPP_RULES); + return ImmutableList.of(CoreRules.INSTANCE, CPP_RULES); } }; @@ -653,7 +628,7 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, CPP_RULES, JAVA_RULES, OBJC_RULES); + return ImmutableList.of(CoreRules.INSTANCE, CPP_RULES, JAVA_RULES, OBJC_RULES); } }; @@ -677,14 +652,14 @@ public class BazelRuleClassProvider { @Override public ImmutableList<RuleSet> requires() { - return ImmutableList.of(CORE_RULES, CORE_WORKSPACE_RULES); + return ImmutableList.of(CoreRules.INSTANCE, CORE_WORKSPACE_RULES); } }; private static final ImmutableSet<RuleSet> RULE_SETS = ImmutableSet.of( BAZEL_SETUP, - CORE_RULES, + CoreRules.INSTANCE, CORE_WORKSPACE_RULES, GENERIC_RULES, CONFIG_RULES, diff --git a/src/main/java/com/google/devtools/build/lib/rules/core/CoreRules.java b/src/main/java/com/google/devtools/build/lib/rules/core/CoreRules.java new file mode 100644 index 0000000000..9494b4edd5 --- /dev/null +++ b/src/main/java/com/google/devtools/build/lib/rules/core/CoreRules.java @@ -0,0 +1,55 @@ +// Copyright 2017 The Bazel Authors. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. +package com.google.devtools.build.lib.rules.core; + +import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableSet; +import com.google.devtools.build.lib.analysis.BaseRuleClasses; +import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider.Builder; +import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider.RuleSet; +import com.google.devtools.build.lib.analysis.featurecontrol.FeaturePolicyLoader; +import com.google.devtools.build.lib.analysis.featurecontrol.FeaturePolicyOptions; +import com.google.devtools.build.lib.rules.config.ConfigFeatureFlag; + +/** A set of basic rules - Bazel won't work correctly without these. */ +public final class CoreRules implements RuleSet { + public static final CoreRules INSTANCE = new CoreRules(); + + public static final ImmutableSet<String> FEATURE_POLICY_FEATURES = + ImmutableSet.of(ConfigFeatureFlag.POLICY_NAME); + + private CoreRules() { + // Use the static INSTANCE field instead. + } + + @Override + public void init(Builder builder) { + builder.addConfigurationOptions(FeaturePolicyOptions.class); + builder.addConfigurationFragment(new FeaturePolicyLoader(FEATURE_POLICY_FEATURES)); + builder.addDynamicTransitionMaps(BaseRuleClasses.DYNAMIC_TRANSITIONS_MAP); + + builder.addRuleDefinition(new BaseRuleClasses.RootRule()); + builder.addRuleDefinition(new BaseRuleClasses.BaseRule()); + builder.addRuleDefinition(new BaseRuleClasses.RuleBase()); + builder.addRuleDefinition(new BaseRuleClasses.MakeVariableExpandingRule()); + builder.addRuleDefinition(new BaseRuleClasses.BinaryBaseRule()); + builder.addRuleDefinition(new BaseRuleClasses.TestBaseRule()); + builder.addRuleDefinition(new BaseRuleClasses.ErrorRule()); + } + + @Override + public ImmutableList<RuleSet> requires() { + return ImmutableList.of(); + } +} |