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

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

Issue 2071573003: GN: Use implicit dependency for binary input deps (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Code review changes Created 4 years, 6 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/ninja_target_writer_unittest.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 1030 matching lines...) Expand 10 before | Expand all | Expand 10 after
1041 " listed in the inputs, the build will detect when it has changed in any\n" 1041 " listed in the inputs, the build will detect when it has changed in any\n"
1042 " way and the action will re-run.\n" 1042 " way and the action will re-run.\n"
1043 "\n" 1043 "\n"
1044 "Inputs for binary targets\n" 1044 "Inputs for binary targets\n"
1045 "\n" 1045 "\n"
1046 " Any input dependencies will be resolved before compiling any sources.\n" 1046 " Any input dependencies will be resolved before compiling any sources.\n"
1047 " Normally, all actions that a target depends on will be run before any\n" 1047 " Normally, all actions that a target depends on will be run before any\n"
1048 " files in a target are compiled. So if you depend on generated headers,\n" 1048 " files in a target are compiled. So if you depend on generated headers,\n"
1049 " you do not typically need to list them in the inputs section.\n" 1049 " you do not typically need to list them in the inputs section.\n"
1050 "\n" 1050 "\n"
1051 " Inputs for binary targets will be treated as order-only dependencies,\n" 1051 " Inputs for binary targets will be treated as implicit dependencies,\n"
1052 " meaning that they will be forced up to date before compiling or\n" 1052 " meaning that changes in any of the inputs will force all sources in\n"
1053 " any files in the target, but changes in the inputs will not\n" 1053 " the target to be recompiled. If an input only applies to a subset of\n"
1054 " necessarily force the target to compile. This is because it is\n" 1054 " source files, you may want to split those into a separate target to\n"
1055 " expected that the compiler will report the precise list of input\n" 1055 " avoid unnecessary recompiles.\n"
1056 " dependencies required to recompile each file once the initial build\n"
1057 " is done.\n"
1058 "\n" 1056 "\n"
1059 "Example\n" 1057 "Example\n"
1060 "\n" 1058 "\n"
1061 " action(\"myscript\") {\n" 1059 " action(\"myscript\") {\n"
1062 " script = \"domything.py\"\n" 1060 " script = \"domything.py\"\n"
1063 " inputs = [ \"input.data\" ]\n" 1061 " inputs = [ \"input.data\" ]\n"
1064 " }\n"; 1062 " }\n";
1065 1063
1066 const char kLdflags[] = "ldflags"; 1064 const char kLdflags[] = "ldflags";
1067 const char kLdflags_HelpShort[] = 1065 const char kLdflags_HelpShort[] =
(...skipping 712 matching lines...) Expand 10 before | Expand all | Expand 10 after
1780 INSERT_VARIABLE(Testonly) 1778 INSERT_VARIABLE(Testonly)
1781 INSERT_VARIABLE(Visibility) 1779 INSERT_VARIABLE(Visibility)
1782 INSERT_VARIABLE(WriteRuntimeDeps) 1780 INSERT_VARIABLE(WriteRuntimeDeps)
1783 } 1781 }
1784 return info_map; 1782 return info_map;
1785 } 1783 }
1786 1784
1787 #undef INSERT_VARIABLE 1785 #undef INSERT_VARIABLE
1788 1786
1789 } // namespace variables 1787 } // namespace variables
OLDNEW
« no previous file with comments | « tools/gn/ninja_target_writer_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698