From 01bfc0c2fc864c24cce3a74d6256c61061718185 Mon Sep 17 00:00:00 2001
From: Jingwen Chen <jingwen@google.com>
Date: Thu, 11 Jul 2019 02:53:39 -0700
Subject: [PATCH] Undo breaking change of hiding android_common behind
 --experimental_google_legacy_api

This should be patched into 0.28.0 to fix https://github.com/bazelbuild/intellij/issues/976

Closes #8860.

PiperOrigin-RevId: 257574106
---
 .../lib/skylarkbuildapi/android/AndroidBootstrap.java | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/android/AndroidBootstrap.java b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/android/AndroidBootstrap.java
index b41e4c1bd592de..de0fbe0215c479 100644
--- a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/android/AndroidBootstrap.java
+++ b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/android/AndroidBootstrap.java
@@ -54,10 +54,13 @@ public AndroidBootstrap(
 
   @Override
   public void addBindingsToBuilder(ImmutableMap.Builder<String, Object> builder) {
-    builder.put(
-        "android_common",
-        FlagGuardedValue.onlyWhenExperimentalFlagIsTrue(
-            FlagIdentifier.EXPERIMENTAL_GOOGLE_LEGACY_API, androidCommon));
+    // TODO: Make an incompatible change flag to hide android_common behind
+    // --experimental_google_legacy_api.
+    // Rationale: android_common module contains commonly used functions used outside of
+    // the Android Starlark migration. Let's not break them without an incompatible
+    // change process.
+    builder.put("android_common", androidCommon);
+
     builder.put(
         ApkInfoApi.NAME,
         FlagGuardedValue.onlyWhenExperimentalFlagIsTrue(