Remove --compile_only and --compilation_prerequisites_only; --output_groups can be used in their stead.

--
MOS_MIGRATED_REVID=87334648

1 files changed, 6 insertions, 51 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java b/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java
index 0b56fa06d8..c9133d0fb3 100644
--- a/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java
+++ b/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java
@@ -24,7 +24,6 @@ import com.google.devtools.build.lib.Constants;
 import com.google.devtools.build.lib.analysis.BuildView;
 import com.google.devtools.build.lib.analysis.TopLevelArtifactContext;
 import com.google.devtools.build.lib.analysis.TopLevelArtifactProvider;
-import com.google.devtools.build.lib.analysis.config.BuildConfiguration.Options;
 import com.google.devtools.build.lib.analysis.config.InvalidConfigurationException;
 import com.google.devtools.build.lib.exec.ExecutionOptions;
 import com.google.devtools.build.lib.pkgcache.LoadingPhaseRunner;
@@ -214,21 +213,6 @@ public class BuildRequest implements OptionsClassProvider {
             + "those phases.")
     public boolean performExecutionPhase;
 
-    @Option(name = "compile_only",
-        defaultValue = "false",
-        category = "what",
-        help = "If specified, Blaze will only build files that are generated by lightweight "
-            + "compilation actions, skipping more expensive build steps (such as linking).")
-    public boolean compileOnly;
-
-    @Option(name = "compilation_prerequisites_only",
-        defaultValue = "false",
-        category = "what",
-        help = "If specified, Blaze will only build files that are prerequisites to compilation "
-             + "of the given target (for example, generated source files and headers) without "
-             + "building the target itself. This flag is ignored if --compile_only is enabled.")
-    public boolean compilationPrerequisitesOnly;
-
     @Option(name = "output_groups",
         converter = Converters.CommaSeparatedOptionListConverter.class,
         allowMultiple = true,
@@ -237,14 +221,6 @@ public class BuildRequest implements OptionsClassProvider {
         help = "Specifies, which output groups of the top-level target to build.")
     public List<String> outputGroups;
 
-    @Option(name = "build_default_artifacts",
-        defaultValue = "true",
-        category = "undocumented",
-        help = "Whether to build the files to build of the configured targets on the command line. "
-            + "If false, only the artifacts specified by --output_groups is built. The default "
-            + "is true.")
-    public boolean buildDefaultArtifacts;
-
     @Option(name = "show_result",
             defaultValue = "1",
             category = "verbosity",
@@ -520,10 +496,6 @@ public class BuildRequest implements OptionsClassProvider {
     if (getBuildOptions().verboseExplanations && getBuildOptions().explanationPath == null) {
       warnings.add("--verbose_explanations has no effect when --explain=<file> is not enabled");
     }
-    if (getBuildOptions().compileOnly && getBuildOptions().compilationPrerequisitesOnly) {
-      throw new InvalidConfigurationException(
-          "--compile_only and --compilation_prerequisites_only are not compatible");
-    }
 
     return warnings;
   }
@@ -540,30 +512,13 @@ public class BuildRequest implements OptionsClassProvider {
     current.add(TopLevelArtifactProvider.TEMP_FILES);
     current.add(TopLevelArtifactProvider.HIDDEN_TOP_LEVEL);
     current.add(TopLevelArtifactProvider.DEFAULT);
-    current.addAll(getBuildOptions().outputGroups);
-
-    if (getOptions(Options.class).collectCodeCoverage && runTests) {
-      current.add(TopLevelArtifactProvider.BASELINE_COVERAGE);
-    }
-
-    BuildRequestOptions options = getOptions(BuildRequestOptions.class);
-    if (!options.buildDefaultArtifacts) {
-      current.remove(TopLevelArtifactProvider.DEFAULT);
-      current.remove(TopLevelArtifactProvider.HIDDEN_TOP_LEVEL);
-    }
-
-    if (options.compileOnly) {
-      current.add(TopLevelArtifactProvider.FILES_TO_COMPILE);
-      current.remove(TopLevelArtifactProvider.BASELINE_COVERAGE);
-      current.remove(TopLevelArtifactProvider.DEFAULT);
-      current.remove(TopLevelArtifactProvider.HIDDEN_TOP_LEVEL);
-    }
 
-    if (options.compilationPrerequisitesOnly) {
-      current.add(TopLevelArtifactProvider.COMPILATION_PREREQUISITES);
-      current.remove(TopLevelArtifactProvider.BASELINE_COVERAGE);
-      current.remove(TopLevelArtifactProvider.DEFAULT);
-      current.remove(TopLevelArtifactProvider.HIDDEN_TOP_LEVEL);
+    for (String outputGroup : getBuildOptions().outputGroups) {
+      if (outputGroup.startsWith("-")) {
+        current.remove(outputGroup.substring(1));
+      } else {
+        current.add(outputGroup);
+      }
     }
 
     return ImmutableSortedSet.copyOf(current);