Remove all traces of the --host_jvm_profile command line argument.

RELNOTES[INC]: The --host_jvm_profile command line argument is not supported anymore. PiperOrigin-RevId: 602952190 Change-Id: I360f875d0783a7f9b8b8cc1ad8e6bd496dab831c
bazelbuild · Jan 31, 2024 · 3491250 · 3491250
1 parent a4c3d89
commit 3491250
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 19 deletions.
diff --git a/src/main/cpp/blaze.cc b/src/main/cpp/blaze.cc
@@ -488,9 +488,6 @@ static vector<string> GetServerExeArgs(const blaze_util::Path &jvm_path,
   if (startup_options.host_jvm_debug) {
     result.push_back("--host_jvm_debug");
   }
-  if (!startup_options.host_jvm_profile.empty()) {
-    result.push_back("--host_jvm_profile=" + startup_options.host_jvm_profile);
-  }
   if (!startup_options.host_jvm_args.empty()) {
     for (const auto &arg : startup_options.host_jvm_args) {
       result.push_back("--host_jvm_args=" + arg);

diff --git a/src/main/cpp/startup_options.cc b/src/main/cpp/startup_options.cc
@@ -277,10 +277,6 @@ blaze_exit_code::ExitCode StartupOptions::ProcessArg(
              nullptr) {
     failure_detail_out = blaze_util::Path(blaze::AbsolutePathFromFlag(value));
     option_sources["failure_detail_out"] = rcfile;
-  } else if ((value = GetUnaryOption(arg, next_arg, "--host_jvm_profile")) !=
-             nullptr) {
-    host_jvm_profile = value;
-    option_sources["host_jvm_profile"] = rcfile;
   } else if ((value = GetUnaryOption(arg, next_arg, "--server_javabase")) !=
              nullptr) {
     // TODO(bazel-team): Consider examining the javabase and re-execing in case

diff --git a/src/main/cpp/startup_options.h b/src/main/cpp/startup_options.h
@@ -167,8 +167,6 @@ class StartupOptions {
 
   bool autodetect_server_javabase;
 
-  std::string host_jvm_profile;
-
   std::vector<std::string> host_jvm_args;
 
   bool batch;

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/HostJvmStartupOptions.java b/src/main/java/com/google/devtools/build/lib/runtime/HostJvmStartupOptions.java
@@ -48,15 +48,6 @@ public class HostJvmStartupOptions extends OptionsBase {
       help = "Flags to pass to the JVM executing Blaze.")
   public List<String> hostJvmArgs;
 
-  @Option(
-      name = "host_jvm_profile",
-      defaultValue = "", // NOTE: purely decorative!  See BlazeServerStartupOptions.
-      valueHelp = "<profiler_name>",
-      documentationCategory = OptionDocumentationCategory.UNCATEGORIZED,
-      effectTags = {OptionEffectTag.NO_OP},
-      help = "Deprecated no-op.")
-  public String hostJvmProfile;
-
   @Option(
     name = "host_jvm_debug",
     defaultValue = "null", // NOTE: purely decorative!  See BlazeServerStartupOptions.

diff --git a/src/test/cpp/bazel_startup_options_test.cc b/src/test/cpp/bazel_startup_options_test.cc
@@ -112,7 +112,6 @@ TEST_F(BazelStartupOptionsTest, ValidStartupFlags) {
   ExpectIsUnaryOption(options, "connect_timeout_secs");
   ExpectIsUnaryOption(options, "digest_function");
   ExpectIsUnaryOption(options, "host_jvm_args");
-  ExpectIsUnaryOption(options, "host_jvm_profile");
   ExpectIsUnaryOption(options, "install_base");
   ExpectIsUnaryOption(options, "invocation_policy");
   ExpectIsUnaryOption(options, "io_nice_level");