Don't load dumpsyms in configurations.

RELNOTES[INC]: Removed --objc_dump_syms_binary.

--
MOS_MIGRATED_REVID=113186554

src/main/java/com/google/devtools/build/lib/rules/objc/CompilationSupport.java

@@ -1223,7 +1223,7 @@ private CompilationSupport registerDsymActions() {
             .addOutput(debugSymbolFile)
             .build(ruleContext));
 
-    Artifact dumpsyms = ruleContext.getPrerequisiteArtifact(":dumpsyms", Mode.HOST);
+    Artifact dumpsyms = ruleContext.getPrerequisiteArtifact("$dumpsyms", Mode.HOST);
     Artifact breakpadFile = intermediateArtifacts.breakpadSym();
     ruleContext.registerAction(ObjcRuleClasses.spawnOnDarwinActionBuilder()
         .setMnemonic("GenBreakpad")

src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCommandLineOptions.java

@@ -16,12 +16,8 @@
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.collect.ImmutableList;
-import com.google.common.collect.Multimap;
-import com.google.devtools.build.lib.Constants;
-import com.google.devtools.build.lib.analysis.config.BuildConfiguration.DefaultLabelConverter;
 import com.google.devtools.build.lib.analysis.config.BuildOptions;
 import com.google.devtools.build.lib.analysis.config.FragmentOptions;
-import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.packages.Attribute.SplitTransition;
 import com.google.devtools.build.lib.rules.apple.DottedVersion;
 import com.google.devtools.build.lib.rules.apple.DottedVersionConverter;
@@ -36,13 +32,6 @@
  * Command-line options for building Objective-C targets.
  */
 public class ObjcCommandLineOptions extends FragmentOptions {
-  /** Converter for --objc_dump_syms_binary. */
-  public static class DumpSymsConverter extends DefaultLabelConverter {
-    public DumpSymsConverter() {
-      super(Constants.TOOLS_REPOSITORY + "//tools/objc:dump_syms");
-    }
-  }
-
   @Option(
     name = "ios_simulator_version",
     defaultValue = "8.4",
@@ -98,12 +87,6 @@ public DumpSymsConverter() {
           "Don't set this value from the command line - it is derived from  ios_multi_cpus only.")
   public String iosSplitCpu;
 
-  @Option(name = "objc_dump_syms_binary",
-      defaultValue = "",
-      category = "undocumented",
-      converter = DumpSymsConverter.class)
-  public Label dumpSyms;
-
   @Option(name = "experimental_enable_objc_cc_deps",
       defaultValue = "false",
       category = "undocumented",
@@ -181,13 +164,6 @@ public DumpSymsConverter() {
 
   @VisibleForTesting static final String DEFAULT_MINIMUM_IOS = "7.0";
 
-  @Override
-  public void addAllLabels(Multimap<String, Label> labelMap) {
-    if (generateDebugSymbols) {
-      labelMap.put("dump_syms", dumpSyms);
-    }
-  }
-
   @Override
   public List<SplitTransition<BuildOptions>> getPotentialSplitTransitions() {
     return ImmutableList.of(

src/main/java/com/google/devtools/build/lib/rules/objc/ObjcConfiguration.java

@@ -70,7 +70,6 @@ public class ObjcConfiguration extends BuildConfiguration.Fragment {
   // configuration, the late bound attribute will fail to be initialized because it hasn't been
   // loaded.
   @Nullable private final Label gcovLabel;
-  @Nullable private final Label dumpSymsLabel;
 
   ObjcConfiguration(ObjcCommandLineOptions objcOptions, BuildConfiguration.Options options,
       @Nullable BlazeDirectories directories) {
@@ -84,7 +83,6 @@ public class ObjcConfiguration extends BuildConfiguration.Fragment {
     this.copts = ImmutableList.copyOf(objcOptions.copts);
     this.compilationMode = Preconditions.checkNotNull(options.compilationMode, "compilationMode");
     this.gcovLabel = options.objcGcovBinary;
-    this.dumpSymsLabel = objcOptions.dumpSyms;
     this.iosSplitCpu = Preconditions.checkNotNull(objcOptions.iosSplitCpu, "iosSplitCpu");
     this.fastbuildOptions = ImmutableList.copyOf(objcOptions.fastbuildOptions);
     this.enableBinaryStripping = objcOptions.enableBinaryStripping;
@@ -163,14 +161,6 @@ public List<String> getCopts() {
     return gcovLabel;
   }
 
-  /**
-   * Returns the label of the dump_syms binary, used to get debug symbols from a binary. Null iff
-   * !{@link #generateDebugSymbols}.
-   */
-  @Nullable public Label getDumpSymsLabel() {
-    return dumpSymsLabel;
-  }
-
   /**
    * Whether module map generation and interpretation is enabled.
    */

src/main/java/com/google/devtools/build/lib/rules/objc/ObjcRuleClasses.java

@@ -776,18 +776,10 @@ public static class LinkingRule implements RuleDefinition {
     @Override
     public RuleClass build(Builder builder, RuleDefinitionEnvironment env) {
       return builder
-          .add(attr(":dumpsyms", LABEL)
+          .add(attr("$dumpsyms", LABEL)
           .cfg(HOST)
           .singleArtifact()
-          .value(new LateBoundLabel<BuildConfiguration>(ObjcConfiguration.class) {
-            @Override
-            public Label getDefault(Rule rule, BuildConfiguration configuration) {
-              if (!configuration.getFragment(ObjcConfiguration.class).generateDebugSymbols()) {
-                return null;
-              }
-              return configuration.getFragment(ObjcConfiguration.class).getDumpSymsLabel();
-            }
-          }))
+          .value(env.getLabel(Constants.TOOLS_REPOSITORY + "//tools/objc:dump_syms")))
           .add(attr("$j2objc_dead_code_pruner", LABEL)
               .allowedFileTypes(FileType.of(".py"))
               .cfg(HOST)

tools/objc/BUILD

@@ -54,6 +54,11 @@ filegroup(
     srcs = [":memleaks_plugin_stub"],
 )
 
+filegroup(
+    name = "dump_syms",
+    srcs = [":dump_syms_stub"],
+)
+
 sh_binary(
     name = "ibtoolwrapper",
     srcs = [":ibtoolwrapper.sh"],

tools/objc/dump_syms_stub

@@ -0,0 +1,7 @@
+#!/bin/bash
+
+(
+  printf 'Bazel does not yet support symbol dumping.\n'
+) >&2
+
+exit 1