Remove topLevelConfigurationHook now that LIPO is gone

PiperOrigin-RevId: 200801973

1 files changed, 0 insertions, 41 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
index a3d59837fa..e9980f8578 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
@@ -37,8 +37,6 @@ import com.google.devtools.build.lib.actions.CommandLines.CommandLineLimits;
 import com.google.devtools.build.lib.analysis.BlazeDirectories;
 import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider;
 import com.google.devtools.build.lib.analysis.actions.FileWriteAction;
-import com.google.devtools.build.lib.analysis.config.transitions.ConfigurationTransition;
-import com.google.devtools.build.lib.analysis.config.transitions.PatchTransition;
 import com.google.devtools.build.lib.buildeventstream.BuildEventId;
 import com.google.devtools.build.lib.buildeventstream.BuildEventStreamProtos;
 import com.google.devtools.build.lib.cmdline.Label;
@@ -48,7 +46,6 @@ import com.google.devtools.build.lib.concurrent.BlazeInterners;
 import com.google.devtools.build.lib.events.Event;
 import com.google.devtools.build.lib.events.EventHandler;
 import com.google.devtools.build.lib.packages.RuleClassProvider;
-import com.google.devtools.build.lib.packages.Target;
 import com.google.devtools.build.lib.packages.TestTimeout;
 import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import com.google.devtools.build.lib.skylarkbuildapi.BuildConfigurationApi;
@@ -162,23 +159,6 @@ public class BuildConfiguration implements BuildConfigurationApi {
     public Map<String, Object> lateBoundOptionDefaults() {
       return ImmutableMap.of();
     }
-
-    /**
-     * Returns an extra transition that should apply to top-level targets in this
-     * configuration. Returns null if no transition is needed.
-     *
-     * <p>Overriders should not change {@link FragmentOptions} not associated with their fragment.
-     *
-     * <p>If multiple fragments specify a transition, they're composed together in a
-     * deterministic but undocumented order (so don't write code expecting a specific order).
-     *
-     * <p>Deprecated. The only known use of this is LIPO, which is on its deathbed.
-     */
-    @Nullable
-    @Deprecated
-    public PatchTransition topLevelConfigurationHook(Target toTarget) {
-      return null;
-    }
   }
 
   public static final Label convertOptionsLabel(String input) throws OptionsParsingException {
@@ -1854,27 +1834,6 @@ public class BuildConfiguration implements BuildConfigurationApi {
     return skylarkVisibleFragments.keySet();
   }
 
-  /**
-   * Returns an extra transition that should apply to top-level targets in this
-   * configuration. Returns null if no transition is needed.
-   */
-  @Nullable
-  public ConfigurationTransition topLevelConfigurationHook(Target toTarget) {
-    ConfigurationTransition currentTransition = null;
-    for (Fragment fragment : fragments.values()) {
-      PatchTransition fragmentTransition = fragment.topLevelConfigurationHook(toTarget);
-      if (fragmentTransition == null) {
-        continue;
-      } else if (currentTransition == null) {
-        currentTransition = fragmentTransition;
-      } else {
-        currentTransition =
-            TransitionResolver.composeTransitions(currentTransition, fragmentTransition);
-      }
-    }
-    return currentTransition;
-  }
-
   public BuildEventId getEventId() {
     return BuildEventId.configurationId(checksum());
   }