Display outputs for "executable" targets in "gn desc".

tools/gn/command_desc.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <set>
 #include <sstream>
 
@@ skipping 320 matching lines @@
   if (target->output_type() == Target::ACTION) {
     // Action, print out outputs, don't apply sources to it.
     for (const auto& elem : target->action_values().outputs().list()) {
       OutputString("  " + elem.AsString() + "\n");
     }
   } else if (target->output_type() == Target::CREATE_BUNDLE) {
     std::vector<SourceFile> output_files;
     target->bundle_data().GetOutputsAsSourceFiles(target->settings(),
                                                   &output_files);
     PrintFileList(output_files, std::string(), true, false);
+  } else if (target->output_type() == Target::EXECUTABLE) {
+    const Tool* tool = target->toolchain()->GetToolForTargetFinalOutput(target);
+
+    std::vector<OutputFile> output_files;
+    SubstitutionWriter::ApplyListToLinkerAsOutputFile(
+        target, tool, tool->outputs(), &output_files);
+
+    std::vector<SourceFile> output_files_as_source_file;
+    for (const OutputFile& output_file : output_files) {
+      output_files_as_source_file.push_back(
+          output_file.AsSourceFile(target->settings()->build_settings()));
+    }
+
+    PrintFileList(output_files_as_source_file, std::string(), true, false);
   } else {

[brettw, 2016/06/29 23:12:09]

Currently the "outputs" section was actually the outputs variable. I don't see
any reason not to expand this, but we should document this specifically in the
"gn desc" help.

In this case, you should remove the condition below and always print the outputs
(excepting maybe source sets and groups). Then this else clause should be an
explicit "ACTION_FOREACH", and your new arm should actually be the else case
(handling all binary targets).

[sdefresne, 2016/07/01 13:12:40]

Done.

     const SubstitutionList& outputs = target->action_values().outputs();
     if (!outputs.required_types().empty()) {
       // Display the pattern and resolved pattern separately, since there are
       // subtitutions used.
       OutputString("  Output pattern\n");
       for (const auto& elem : outputs.list())
         OutputString("    " + elem.AsString() + "\n");
 
       // Now display what that resolves to given the sources.
       OutputString("\n  Resolved output file list\n");
@@ skipping 257 matching lines @@
     if (what.empty() || what == variables::kDepfile) {
       PrintDepfile(target, display_headers);
       found_match = true;
     }
   }
 
   // Outputs.
   if (target->output_type() == Target::ACTION ||
       target->output_type() == Target::ACTION_FOREACH ||
       target->output_type() == Target::COPY_FILES ||
-      target->output_type() == Target::CREATE_BUNDLE) {
+      target->output_type() == Target::CREATE_BUNDLE ||
+      target->output_type() == Target::EXECUTABLE) {
     if (what.empty() || what == variables::kOutputs) {
       PrintOutputs(target, display_headers);
       found_match = true;
     }
   }
 
   // Values from configs only apply to binary targets.
   if (is_binary_output) {
     #define CONFIG_VALUE_ARRAY_HANDLER(name, type) \
       if (what.empty() || what == #name) { \
@@ skipping 294 matching lines @@
     printed_output = true;
 
     if (!PrintConfig(config, what_to_print, display_item_header))
       return 1;
   }
 
   return 0;
 }
 
 }  // namespace commands