From 10a6130dbafdf4e5f61198bec08b88fe6d6381fe Mon Sep 17 00:00:00 2001
From: gregce <gregce@google.com>
Date: Fri, 25 May 2018 13:20:05 -0700
Subject: Skylark: Make 'cfg = "data"' a noop.

This is no longer meaningful with the turndown of
(C++) LIPO.

PiperOrigin-RevId: 198092974
---
 .../com/google/devtools/build/lib/analysis/skylark/SkylarkAttr.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

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

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkAttr.java b/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkAttr.java
index a7ecf893e9..94f893ad8b 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkAttr.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkAttr.java
@@ -20,7 +20,6 @@ import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Iterables;
 import com.google.devtools.build.lib.analysis.config.HostTransition;
-import com.google.devtools.build.lib.analysis.config.transitions.PatchTransition;
 import com.google.devtools.build.lib.analysis.config.transitions.SplitTransition;
 import com.google.devtools.build.lib.events.Location;
 import com.google.devtools.build.lib.packages.Attribute;
@@ -45,7 +44,6 @@ import com.google.devtools.build.lib.syntax.SkylarkCallbackFunction;
 import com.google.devtools.build.lib.syntax.SkylarkDict;
 import com.google.devtools.build.lib.syntax.SkylarkList;
 import com.google.devtools.build.lib.syntax.SkylarkType;
-import com.google.devtools.build.lib.syntax.SkylarkUtils;
 import com.google.devtools.build.lib.syntax.Type;
 import com.google.devtools.build.lib.syntax.Type.ConversionException;
 import com.google.devtools.build.lib.syntax.Type.LabelClass;
@@ -226,7 +224,9 @@ public final class SkylarkAttr implements SkylarkAttrApi {
     if (containsNonNoneKey(arguments, CONFIGURATION_ARG)) {
       Object trans = arguments.get(CONFIGURATION_ARG);
       if (trans.equals("data")) {
-        builder.cfg((PatchTransition) SkylarkUtils.getLipoDataTransition(env));
+        // This used to apply the "disable LIPO" (a.k.a. "data") transition. But now that LIPO is
+        // turned down this is a noop. Still, there are cfg = "data"' references in the depot. So
+        // we have to remove them via b/28688645 before we can remove this path.
       } else if (trans.equals("host")) {
         builder.cfg(HostTransition.INSTANCE);
       } else if (trans instanceof SplitTransition) {
-- 
cgit v1.2.3