GN: Don't write ldflags and libs when unneeded.

tools/gn/variables.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 "tools/gn/variables.h"
 
 namespace variables {
 
 // Built-in variables ----------------------------------------------------------
 
@@ skipping 919 matching lines @@
     "  lib_dirs = [ \"/usr/lib/foo\", \"lib/doom_melon\" ]\n";
 
 const char kLibs[] = "libs";
 const char kLibs_HelpShort[] =
     "libs: [string list] Additional libraries to link.";
 const char kLibs_Help[] =
     "libs: Additional libraries to link.\n"
     "\n"
     "  A list of library names or library paths.\n"
     "\n"
-    "Values containing '/' will be treated as references to files in the\n"
-    "build. They will be rebased to be relative to the build directory and\n"
-    "specified in the \"libs\" for linker tools. This facility should be used\n"
-    "for libraries that are checked in to the build. For libraries that are\n"
-    "generated by the build, use normal GN deps to link them.\n"
+    "  These libraries will be linked into the final binary (executable or\n"
+    "  shared library) containing the current target.\n"
+    COMMON_LIB_INHERITANCE_HELP
     "\n"
-    "Values not containing '/' will be treated as system library names. These\n"
-    "will be passed unmodified to the linker and prefixed with the\n"
-    "\"lib_prefix\" attribute of the linker tool. Generally you would set the\n"
-    "\"lib_dirs\" so the given library is found. Your BUILD.gn file should\n"
-    "not specify the switch prefix (like \"-l\"): this will be encoded in\n"
-    "the \"lib_prefix\" of the tool.\n"
+    "Types of libs\n"
     "\n"
-    "  Libraries ending in \".framework\" will be special-cased: the switch\n"
-    "  \"-framework\" will be prepended instead of the lib_prefix, and the\n"
-    "  \".framework\" suffix will be trimmed. This is to support the way Mac\n"
-    "  links framework dependencies.\n"
-    COMMON_LIB_INHERITANCE_HELP
+    "  There are several different things that can be expressed in libs:\n"
+    "\n"
+    "  File paths\n"
+    "      Values containing '/' will be treated as references to files in\n"
+    "      the checkout. They will be rebased to be relative to the build\n"
+    "      directory and specified in the \"libs\" for linker tools. This\n"
+    "      facility should be used for libraries that are checked in to the\n"
+    "      version control. For libraries that are generated by the build,\n"
+    "      use normal GN deps to link them.\n"
+    "\n"
+    "  System libraries\n"
+    "      Values not containing '/' will be treated as system library names.\n"
+    "      These will be passed unmodified to the linker and prefixed with\n"
+    "      the \"lib_prefix\" attribute of the linker tool. Generally you\n"
+    "      would set the \"lib_dirs\" so the given library is found. Your\n"
+    "      BUILD.gn file should not specify the switch (like \"-l\"): this\n"
+    "      will be encoded in the \"lib_prefix\" of the tool.\n"
+    "\n"
+    "  Apple frameworks\n"
+    "      System libraries ending in \".framework\" will be special-cased:\n"
+    "      the switch \"-framework\" will be prepended instead of the\n"
+    "      lib_prefix, and the \".framework\" suffix will be trimmed. This is\n"
+    "      to support the way Mac links framework dependencies.\n"
     COMMON_ORDERING_HELP
     LIBS_AND_LIB_DIRS_ORDERING_HELP
     "\n"
     "Examples\n"
     "\n"
     "  On Windows:\n"
     "    libs = [ \"ctl3d.lib\" ]\n"
+    "\n"
     "  On Linux:\n"
     "    libs = [ \"ld\" ]\n";
 
 const char kOutputExtension[] = "output_extension";
 const char kOutputExtension_HelpShort[] =
     "output_extension: [string] Value to use for the output's file extension.";
 const char kOutputExtension_Help[] =
     "output_extension: Value to use for the output's file extension.\n"
     "\n"
     "  Normally the file extension for a target is based on the target\n"
@@ skipping 501 matching lines @@
     INSERT_VARIABLE(Sources)
     INSERT_VARIABLE(Testonly)
     INSERT_VARIABLE(Visibility)
   }
   return info_map;
 }
 
 #undef INSERT_VARIABLE
 
 }  // namespace variables