From ea2ecbf8c0601b5a1a72dfb849dc1b86a72df533 Mon Sep 17 00:00:00 2001
From: lpino <lpino@google.com>
Date: Tue, 24 Oct 2017 16:03:00 +0200
Subject: PiperOrigin-RevId: 173251680

---
 .../lib/bazel/rules/BazelRuleClassProvider.java    | 75 +++++++++-------------
 1 file changed, 31 insertions(+), 44 deletions(-)

(limited to 'src/main')

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 b2f18dddaa..a0049989d2 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
@@ -353,51 +353,38 @@ public class BazelRuleClassProvider {
         }
       };
 
-  /**
-   * Set the label for Windows DEF parser. In bazel, it should be
-   * @bazel_tools//tools/def_parser:def_parser, otherwise it should be null.
-   *
-   * <p>TODO(pcloudy): Remove this after Bazel rule definitions are not used internally anymore.
-   * Related bug b/63658220
-   */
-  public static final RuleSet CPP_RULES = cppRules("@bazel_tools//tools/def_parser:def_parser");
-
-  public static RuleSet cppRules() {
-    return cppRules(null);
-  }
+  public static final RuleSet CPP_RULES =
+      new RuleSet() {
+        @Override
+        public void init(Builder builder) {
+          builder.addConfig(
+              CppOptions.class, new CppConfigurationLoader(Functions.<String>identity()));
 
-  public static RuleSet cppRules(String defParserLabel) {
-    return new RuleSet() {
-      @Override
-      public void init(Builder builder) {
-        builder.addConfig(
-            CppOptions.class, new CppConfigurationLoader(Functions.<String>identity()));
-
-        builder.addBuildInfoFactory(new CppBuildInfo());
-        builder.addDynamicTransitionMaps(CppRuleClasses.DYNAMIC_TRANSITIONS_MAP);
-
-        builder.addRuleDefinition(new CcToolchainRule(defParserLabel));
-        builder.addRuleDefinition(new CcToolchainSuiteRule());
-        builder.addRuleDefinition(new CcToolchainAlias.CcToolchainAliasRule());
-        builder.addRuleDefinition(new CcIncLibraryRule());
-        builder.addRuleDefinition(new BazelCppRuleClasses.CcLinkingRule());
-        builder.addRuleDefinition(new BazelCppRuleClasses.CcDeclRule());
-        builder.addRuleDefinition(new BazelCppRuleClasses.CcBaseRule());
-        builder.addRuleDefinition(new BazelCppRuleClasses.CcRule());
-        builder.addRuleDefinition(new BazelCppRuleClasses.CcBinaryBaseRule());
-        builder.addRuleDefinition(new BazelCcBinaryRule());
-        builder.addRuleDefinition(new BazelCcTestRule());
-        builder.addRuleDefinition(new BazelCppRuleClasses.CcLibraryBaseRule());
-        builder.addRuleDefinition(new BazelCcLibraryRule());
-        builder.addRuleDefinition(new BazelCcIncLibraryRule());
-      }
-
-      @Override
-      public ImmutableList<RuleSet> requires() {
-        return ImmutableList.of(CoreRules.INSTANCE, PLATFORM_RULES);
-      }
-    };
-  }
+          builder.addBuildInfoFactory(new CppBuildInfo());
+          builder.addDynamicTransitionMaps(CppRuleClasses.DYNAMIC_TRANSITIONS_MAP);
+
+          builder.addRuleDefinition(
+              new CcToolchainRule("@bazel_tools//tools/def_parser:def_parser"));
+          builder.addRuleDefinition(new CcToolchainSuiteRule());
+          builder.addRuleDefinition(new CcToolchainAlias.CcToolchainAliasRule());
+          builder.addRuleDefinition(new CcIncLibraryRule());
+          builder.addRuleDefinition(new BazelCppRuleClasses.CcLinkingRule());
+          builder.addRuleDefinition(new BazelCppRuleClasses.CcDeclRule());
+          builder.addRuleDefinition(new BazelCppRuleClasses.CcBaseRule());
+          builder.addRuleDefinition(new BazelCppRuleClasses.CcRule());
+          builder.addRuleDefinition(new BazelCppRuleClasses.CcBinaryBaseRule());
+          builder.addRuleDefinition(new BazelCcBinaryRule());
+          builder.addRuleDefinition(new BazelCcTestRule());
+          builder.addRuleDefinition(new BazelCppRuleClasses.CcLibraryBaseRule());
+          builder.addRuleDefinition(new BazelCcLibraryRule());
+          builder.addRuleDefinition(new BazelCcIncLibraryRule());
+        }
+
+        @Override
+        public ImmutableList<RuleSet> requires() {
+          return ImmutableList.of(CoreRules.INSTANCE, PLATFORM_RULES);
+        }
+      };
 
   public static final RuleSet CPP_PROTO_RULES =
       new RuleSet() {
-- 
cgit v1.2.3