From 1bb58da17c092288e351185385799d3902116d6f Mon Sep 17 00:00:00 2001
From: Chris Parsons <cparsons@google.com>
Date: Tue, 30 Aug 2022 13:37:57 -0400
Subject: [PATCH] Remove USE_BAZEL_ANALYSIS

Use --bazel-mode-dev instead.

Test: Treehugger
Bug: 244309479
Change-Id: I570d02651eea92375cd96efbeb8d986a9495e1ed
---
 android/bazel.go        |  2 +-
 ui/build/build.go       | 11 +++--------
 ui/build/config_test.go | 10 ----------
 3 files changed, 4 insertions(+), 19 deletions(-)

diff --git a/android/bazel.go b/android/bazel.go
index 27ec92bd1..eb6aca4e9 100644
--- a/android/bazel.go
+++ b/android/bazel.go
@@ -75,7 +75,7 @@ type bazelModuleProperties struct {
 
 // Properties contains common module properties for Bazel migration purposes.
 type properties struct {
-	// In USE_BAZEL_ANALYSIS=1 mode, this represents the Bazel target replacing
+	// In "Bazel mixed build" mode, this represents the Bazel target replacing
 	// this Soong module.
 	Bazel_module bazelModuleProperties
 }
diff --git a/ui/build/build.go b/ui/build/build.go
index ecf11ccd6..ff2d5f28d 100644
--- a/ui/build/build.go
+++ b/ui/build/build.go
@@ -112,17 +112,12 @@ const (
 // checkBazelMode fails the build if there are conflicting arguments for which bazel
 // build mode to use.
 func checkBazelMode(ctx Context, config Config) {
-	// TODO(cparsons): Remove USE_BAZEL_ANALYSIS handling.
 	if config.Environment().IsEnvTrue("USE_BAZEL_ANALYSIS") {
-		if config.bazelProdMode || config.bazelDevMode {
-			ctx.Fatalf("USE_BAZEL_ANALYSIS is deprecated.\n" +
-				"Unset USE_BAZEL_ANALYSIS when using --bazel-mode or --bazel-mode-dev.")
-		} else {
-			config.bazelDevMode = true
-		}
+		ctx.Fatalln("USE_BAZEL_ANALYSIS is deprecated. Unset USE_BAZEL_ANALYSIS.\n" +
+			"Use --bazel-mode-dev instead. For example: `m --bazel-mode-dev nothing`")
 	}
 	if config.bazelProdMode && config.bazelDevMode {
-		ctx.Fatalf("Conflicting bazel mode.\n" +
+		ctx.Fatalln("Conflicting bazel mode.\n" +
 			"Do not specify both --bazel-mode and --bazel-mode-dev")
 	}
 }
diff --git a/ui/build/config_test.go b/ui/build/config_test.go
index 2887f59a6..150ec35dc 100644
--- a/ui/build/config_test.go
+++ b/ui/build/config_test.go
@@ -1061,16 +1061,6 @@ func TestBuildConfig(t *testing.T) {
 				BazelMixedBuild: proto.Bool(false),
 			},
 		},
-		{
-			name:    "bazel mixed build from env",
-			environ: Environment{"USE_BAZEL_ANALYSIS=1"},
-			expectedBuildConfig: &smpb.BuildConfig{
-				ForceUseGoma:    proto.Bool(false),
-				UseGoma:         proto.Bool(false),
-				UseRbe:          proto.Bool(false),
-				BazelMixedBuild: proto.Bool(true),
-			},
-		},
 		{
 			name:         "bazel mixed build from dev mode",
 			environ:      Environment{},