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

Unified Diff: tools/gn/functions_target.cc

Issue 1375023003: tools/gn: Remove code for forward_dependent_configs_from variable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: forward_dependent_configs Created 5 years, 2 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
Index: tools/gn/functions_target.cc
diff --git a/tools/gn/functions_target.cc b/tools/gn/functions_target.cc
index ffe8f5f2941ebfc4bb8d978b244d65a9d71118f5..fbac2d2bfe26c606d013a6820f401e23ba45bf48 100644
--- a/tools/gn/functions_target.cc
+++ b/tools/gn/functions_target.cc
@@ -16,7 +16,7 @@
#define DEPENDENT_CONFIG_VARS \
" Dependent configs: all_dependent_configs, public_configs\n"
#define DEPS_VARS \
- " Deps: data_deps, deps, forward_dependent_configs_from, public_deps\n"
+ " Deps: data_deps, deps, public_deps\n"
#define GENERAL_TARGET_VARS \
" General: check_includes, configs, data, inputs, output_name,\n" \
" output_extension, public, sources, testonly, visibility\n"
@@ -330,9 +330,7 @@ const char kGroup_Help[] =
" specify configs that apply to their dependents.\n"
"\n"
" Depending on a group is exactly like depending directly on that\n"
- " group's deps. Direct dependent configs will get automatically\n"
- " forwarded through the group so you shouldn't need to use\n"
- " \"forward_dependent_configs_from.\n"
+ " group's deps. \n"
"\n"
"Variables\n"
"\n"

Powered by Google App Engine
This is Rietveld 408576698