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

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

Issue 1750283003: Minor fixes to tools/gn. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix last reference to IsValidToolSubstutition. Created 4 years, 9 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
« tools/gn/substitution_type.cc ('K') | « tools/gn/toolchain.h ('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 806 matching lines...) Expand 10 before | Expand all | Expand 10 after
817 " Inputs are compile-time dependencies of the current target. This means\n" 817 " Inputs are compile-time dependencies of the current target. This means\n"
818 " that all inputs must be available before compiling any of the sources\n" 818 " that all inputs must be available before compiling any of the sources\n"
819 " or executing any actions.\n" 819 " or executing any actions.\n"
820 "\n" 820 "\n"
821 " Inputs are typically only used for action and action_foreach targets.\n" 821 " Inputs are typically only used for action and action_foreach targets.\n"
822 "\n" 822 "\n"
823 "Inputs for actions\n" 823 "Inputs for actions\n"
824 "\n" 824 "\n"
825 " For action and action_foreach targets, inputs should be the inputs to\n" 825 " For action and action_foreach targets, inputs should be the inputs to\n"
826 " script that don't vary. These should be all .py files that the script\n" 826 " script that don't vary. These should be all .py files that the script\n"
827 " uses via imports (the main script itself will be an implcit dependency\n" 827 " uses via imports (the main script itself will be an implicit dependency"
828 "\n"
828 " of the action so need not be listed).\n" 829 " of the action so need not be listed).\n"
829 "\n" 830 "\n"
830 " For action targets, inputs and sources are treated the same, but from\n" 831 " For action targets, inputs and sources are treated the same, but from\n"
831 " a style perspective, it's recommended to follow the same rule as\n" 832 " a style perspective, it's recommended to follow the same rule as\n"
832 " action_foreach and put helper files in the inputs, and the data used\n" 833 " action_foreach and put helper files in the inputs, and the data used\n"
833 " by the script (if any) in sources.\n" 834 " by the script (if any) in sources.\n"
834 "\n" 835 "\n"
835 " Note that another way to declare input dependencies from an action\n" 836 " Note that another way to declare input dependencies from an action\n"
836 " is to have the action write a depfile (see \"gn help depfile\"). This\n" 837 " is to have the action write a depfile (see \"gn help depfile\"). This\n"
837 " allows the script to dynamically write input dependencies, that might\n" 838 " allows the script to dynamically write input dependencies, that might\n"
(...skipping 650 matching lines...) Expand 10 before | Expand all | Expand 10 after
1488 INSERT_VARIABLE(Sources) 1489 INSERT_VARIABLE(Sources)
1489 INSERT_VARIABLE(Testonly) 1490 INSERT_VARIABLE(Testonly)
1490 INSERT_VARIABLE(Visibility) 1491 INSERT_VARIABLE(Visibility)
1491 } 1492 }
1492 return info_map; 1493 return info_map;
1493 } 1494 }
1494 1495
1495 #undef INSERT_VARIABLE 1496 #undef INSERT_VARIABLE
1496 1497
1497 } // namespace variables 1498 } // namespace variables
OLDNEW
« tools/gn/substitution_type.cc ('K') | « tools/gn/toolchain.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698