commit a7b34a1591cfa1034dca9b2c4ff9743d81396a97
author Klaus Aehlig <aehlig@google.com> Tue Feb 20 09:31:37 2018 -0800
committer Copybara-Service <copybara-piper@google.com> Tue Feb 20 09:35:06 2018 -0800
tree 5626845e44389dd5b5758b8f466368337dba8fed
parent cc16e7cb3e7203f5033928f1348722cad6d8e59d

Decprecate native http_archive

Since the skylark implementation of http_archive provides all
the features available in the native http_archive and a few
more (patching), there is no need to have to rule implementations
doing the same thing. Hence deprecate the native one as part
of our plan of moving more functionality to skylark.

RELNOTES: The native http_archive rule is deprecated. Use the
  Skylark version available via load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
  instead.

Change-Id: I107c2f25f5a37c67f56b4362e7c9d9ade8428c16
PiperOrigin-RevId: 186314624

src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java
index ccdef1e..d14a09c 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java
@@ -52,6 +52,9 @@
   public abstract boolean incompatibleLoadArgumentIsLabel();
   public abstract boolean incompatibleNewActionsApi();
   public abstract boolean incompatibleShowAllPrintMessages();
+
+  public abstract boolean incompatibleRemoveNativeHttpArchive();
+
   public abstract boolean incompatibleStringIsNotIterable();
   public abstract boolean internalSkylarkFlagTestCanary();
 
@@ -79,10 +82,11 @@
           .incompatibleDisallowUncalledSetConstructor(true)
           .incompatibleLoadArgumentIsLabel(true)
           .incompatibleNewActionsApi(false)
+          .incompatibleRemoveNativeHttpArchive(false)
           .incompatibleShowAllPrintMessages(true)
-      .incompatibleStringIsNotIterable(false)
-      .internalSkylarkFlagTestCanary(false)
-      .build();
+          .incompatibleStringIsNotIterable(false)
+          .internalSkylarkFlagTestCanary(false)
+          .build();
 
   /** Builder for {@link SkylarkSemantics}. All fields are mandatory. */
   @AutoValue.Builder
@@ -101,6 +105,9 @@
     public abstract Builder incompatibleDisallowUncalledSetConstructor(boolean value);
     public abstract Builder incompatibleLoadArgumentIsLabel(boolean value);
     public abstract Builder incompatibleNewActionsApi(boolean value);
+
+    public abstract Builder incompatibleRemoveNativeHttpArchive(boolean value);
+
     public abstract Builder incompatibleShowAllPrintMessages(boolean value);
     public abstract Builder incompatibleStringIsNotIterable(boolean value);
     public abstract Builder internalSkylarkFlagTestCanary(boolean value);