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

Unified Diff: tools/gn/variables.cc

Issue 1130183007: Add runtime dependency extraction for GN. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: make msvc happy Created 5 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/gn/switches.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/gn/variables.cc
diff --git a/tools/gn/variables.cc b/tools/gn/variables.cc
index 4d339f076661c0a8df0bcc06a760fb17e0bd30ae..1cbfc9cfb9b894b512ee7ae28b8a6908bc72a915 100644
--- a/tools/gn/variables.cc
+++ b/tools/gn/variables.cc
@@ -510,16 +510,23 @@ const char kData_Help[] =
"data: Runtime data file dependencies.\n"
"\n"
" Lists files required to run the given target. These are typically\n"
- " data files.\n"
+ " data files. The paths are interpreted as being relative to the current\n"
+ " build file. Since these are runtime dependencies, they do not affect\n"
+ " which targets are built or when. To declare input files to a script,\n"
+ " use \"inputs\".\n"
"\n"
" Appearing in the \"data\" section does not imply any special handling\n"
" such as copying them to the output directory. This is just used for\n"
- " declaring runtime dependencies. There currently isn't a good use for\n"
- " these but it is envisioned that test data can be listed here for use\n"
- " running automated tests.\n"
+ " declaring runtime dependencies. Runtime dependencies can be queried\n"
+ " using the \"runtime_deps\" category of \"gn desc\" or written during\n"
+ " build generation via \"--runtime-deps-list-file\".\n"
"\n"
- " See also \"gn help inputs\" and \"gn help data_deps\", both of\n"
- " which actually affect the build in concrete ways.\n";
+ " GN doesn't require data files to exist at build-time. So actions that\n"
+ " produce files that are in turn runtime dependencies can list those\n"
+ " generated files both in the \"outputs\" list as well as the \"data\"\n"
+ " list.\n"
+ "\n"
+ " See \"gn help runtime_deps\" for how these are used.\n";
const char kDataDeps[] = "data_deps";
const char kDataDeps_HelpShort[] =
« no previous file with comments | « tools/gn/switches.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698