From f223689843a37268d0dd338aa10c94d0626d6e23 Mon Sep 17 00:00:00 2001
From: Lukacs Berki <lberki@google.com>
Date: Tue, 31 May 2016 08:01:36 +0000
Subject: Remove the --use_isystem_for_includes option.

RELNOTES: The --ues_isystem_for_includes command line option is not supported anymore.

--
MOS_MIGRATED_REVID=123617061
---
 .../build/lib/bazel/rules/cpp/BazelCppRuleClasses.java  |  4 ----
 .../google/devtools/build/lib/rules/cpp/CcCommon.java   | 17 -----------------
 .../devtools/build/lib/rules/cpp/CcLibraryHelper.java   |  1 -
 .../devtools/build/lib/rules/cpp/CppConfiguration.java  |  4 ----
 .../google/devtools/build/lib/rules/cpp/CppOptions.java | 10 ----------
 5 files changed, 36 deletions(-)

(limited to 'src/main/java/com/google/devtools')

diff --git a/src/main/java/com/google/devtools/build/lib/bazel/rules/cpp/BazelCppRuleClasses.java b/src/main/java/com/google/devtools/build/lib/bazel/rules/cpp/BazelCppRuleClasses.java
index fc36c17f98..800b95ad88 100644
--- a/src/main/java/com/google/devtools/build/lib/bazel/rules/cpp/BazelCppRuleClasses.java
+++ b/src/main/java/com/google/devtools/build/lib/bazel/rules/cpp/BazelCppRuleClasses.java
@@ -313,10 +313,6 @@ public class BazelCppRuleClasses {
              very careful, since this may have far-reaching effects.  When in doubt, add
              "-I" flags to <a href="#cc_binary.copts">COPTS</a> instead.
           </p>
-          <p>To use <code>-iquote</code> instead of <code>-isystem</code>, specify
-             <code>--use_isystem_for_includes=false</code> (the flag is undocumented and defaults
-             to <code>true</code>).
-          </p>
           <!-- #END_BLAZE_RULE.ATTRIBUTE -->*/
           .add(attr("includes", STRING_LIST))
           .add(attr(":lipo_context_collector", LABEL)
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java
index 44623f6974..a7b1c3f64b 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCommon.java
@@ -384,23 +384,6 @@ public final class CcCommon {
   }
 
   List<PathFragment> getSystemIncludeDirs() {
-    // Add in any 'includes' attribute values as relative path fragments
-    if (!ruleContext.getRule().isAttributeValueExplicitlySpecified("includes")
-        || !cppConfiguration.useIsystemForIncludes()) {
-      return ImmutableList.of();
-    }
-    return getIncludeDirsFromIncludesAttribute();
-  }
-
-  List<PathFragment> getIncludeDirs() {
-    if (!ruleContext.getRule().isAttributeValueExplicitlySpecified("includes")
-        || cppConfiguration.useIsystemForIncludes()) {
-      return ImmutableList.of();
-    }
-    return getIncludeDirsFromIncludesAttribute();
-  }
-
-  private List<PathFragment> getIncludeDirsFromIncludesAttribute() {
     List<PathFragment> result = new ArrayList<>();
     PackageIdentifier packageIdentifier = ruleContext.getLabel().getPackageIdentifier();
     PathFragment packageFragment = packageIdentifier.getPathFragment();
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java
index a3ec1d01c0..d4c4da31e6 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibraryHelper.java
@@ -310,7 +310,6 @@ public final class CcLibraryHelper {
         .addCopts(common.getCopts())
         .addDefines(common.getDefines())
         .addDeps(ruleContext.getPrerequisites("deps", Mode.TARGET))
-        .addIncludeDirs(common.getIncludeDirs())
         .addLooseIncludeDirs(common.getLooseIncludeDirs())
         .addPicIndependentObjectFiles(common.getLinkerScripts())
         .addSystemIncludeDirs(common.getSystemIncludeDirs())
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java
index 2fc10e733b..319610243b 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppConfiguration.java
@@ -1677,10 +1677,6 @@ public class CppConfiguration extends BuildConfiguration.Fragment {
     return cppOptions.inmemoryDotdFiles;
   }
 
-  public boolean useIsystemForIncludes() {
-    return cppOptions.useIsystemForIncludes;
-  }
-
   public LibcTop getLibcTop() {
     return cppOptions.libcTop;
   }
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppOptions.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppOptions.java
index 931db76d9a..dd1b3da2a7 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppOptions.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppOptions.java
@@ -540,16 +540,6 @@ public class CppOptions extends FragmentOptions {
   )
   public boolean inmemoryDotdFiles;
 
-  @Option(
-    name = "use_isystem_for_includes",
-    defaultValue = "true",
-    category = "undocumented",
-    help =
-        "Instruct C and C++ compilations to treat 'includes' paths as system header "
-            + "paths, by translating it into -isystem instead of -I."
-  )
-  public boolean useIsystemForIncludes;
-
   @Option(
     name = "experimental_omitfp",
     defaultValue = "false",
-- 
cgit v1.2.3