Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(27)

Side by Side Diff: tools/gn/variables.cc

Issue 1314773005: Throw errors for nested targets in GN. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tools/gn/functions_target.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "tools/gn/variables.h" 5 #include "tools/gn/variables.h"
6 6
7 namespace variables { 7 namespace variables {
8 8
9 // Built-in variables ---------------------------------------------------------- 9 // Built-in variables ----------------------------------------------------------
10 10
(...skipping 749 matching lines...) Expand 10 before | Expand all | Expand 10 after
760 "\n" 760 "\n"
761 " A list of strings.\n" 761 " A list of strings.\n"
762 "\n" 762 "\n"
763 " These flags are passed on the command-line to the linker and generally\n" 763 " These flags are passed on the command-line to the linker and generally\n"
764 " specify various linking options. Most targets will not need these and\n" 764 " specify various linking options. Most targets will not need these and\n"
765 " will use \"libs\" and \"lib_dirs\" instead.\n" 765 " will use \"libs\" and \"lib_dirs\" instead.\n"
766 "\n" 766 "\n"
767 " ldflags are NOT pushed to dependents, so applying ldflags to source\n" 767 " ldflags are NOT pushed to dependents, so applying ldflags to source\n"
768 " sets or static libraries will be a no-op. If you want to apply ldflags\n" 768 " sets or static libraries will be a no-op. If you want to apply ldflags\n"
769 " to dependent targets, put them in a config and set it in the\n" 769 " to dependent targets, put them in a config and set it in the\n"
770 " all_dependent_configs or public_configs.\n"; 770 " all_dependent_configs or public_configs.\n"
771 COMMON_ORDERING_HELP;
771 772
772 #define COMMON_LIB_INHERITANCE_HELP \ 773 #define COMMON_LIB_INHERITANCE_HELP \
773 "\n" \ 774 "\n" \
774 " libs and lib_dirs work differently than other flags in two respects.\n" \ 775 " libs and lib_dirs work differently than other flags in two respects.\n" \
775 " First, then are inherited across static library boundaries until a\n" \ 776 " First, then are inherited across static library boundaries until a\n" \
776 " shared library or executable target is reached. Second, they are\n" \ 777 " shared library or executable target is reached. Second, they are\n" \
777 " uniquified so each one is only passed once (the first instance of it\n" \ 778 " uniquified so each one is only passed once (the first instance of it\n" \
778 " will be the one used).\n" 779 " will be the one used).\n"
779 780
780 const char kLibDirs[] = "lib_dirs"; 781 const char kLibDirs[] = "lib_dirs";
(...skipping 440 matching lines...) Expand 10 before | Expand all | Expand 10 after
1221 INSERT_VARIABLE(Sources) 1222 INSERT_VARIABLE(Sources)
1222 INSERT_VARIABLE(Testonly) 1223 INSERT_VARIABLE(Testonly)
1223 INSERT_VARIABLE(Visibility) 1224 INSERT_VARIABLE(Visibility)
1224 } 1225 }
1225 return info_map; 1226 return info_map;
1226 } 1227 }
1227 1228
1228 #undef INSERT_VARIABLE 1229 #undef INSERT_VARIABLE
1229 1230
1230 } // namespace variables 1231 } // namespace variables
OLDNEW
« no previous file with comments | « tools/gn/functions_target.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698