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

tools/gn/command_desc.cc

Index: tools/gn/command_desc.cc
diff --git a/tools/gn/command_desc.cc b/tools/gn/command_desc.cc
index d1edfe487b42baadd4e1cdfac511125b1a582a93..00224f2cf3a97523fb8a8d9bc557efa556e43dec 100644
--- a/tools/gn/command_desc.cc
+++ b/tools/gn/command_desc.cc
@@ -338,6 +338,20 @@ void PrintOutputs(const Target* target, bool display_header) {
     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()) {
@@ -616,7 +630,8 @@ bool PrintTarget(const Target* target,
   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;