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

Unified Diff: testing/test.gni

Issue 1280983003: Forward "include_dirs" to executable target in "ios_app" template. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@fix-ios-release
Patch Set: Move the forward_variables_from to the correct target Created 5 years, 4 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/test.gni
diff --git a/testing/test.gni b/testing/test.gni
index 28a9e550df84865a6d01ef77af4be1e2cc721414..3780ebd5f757548f65409816abc8e94c5c1ab8bb 100644
--- a/testing/test.gni
+++ b/testing/test.gni
@@ -121,11 +121,17 @@ template("test") {
forward_variables_from(invoker,
[
+ "cflags",
+ "cflags_c",
+ "cflags_cc",
+ "cflags_objc",
+ "cflags_objcc",
"data",
- "deps",
"data_deps",
"datadeps",
"defines",
+ "deps",
+ "include_dirs",
"includes",
])
}
@@ -154,7 +160,6 @@ template("test") {
"cflags_objcc",
"check_includes",
"forward_dependent_configs_from",
- "include_dirs",
"ldflags",
"libs",
"output_extension",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698