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

Unified Diff: chrome/version.gni

Issue 668343004: Make channel available in Java via ChromeVersionConstants.java. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix GN build Created 6 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
« chrome/android/channel.gni ('K') | « chrome/chrome_android.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/version.gni
diff --git a/chrome/version.gni b/chrome/version.gni
index 771ea28541d39b6ea0373a6485e6132622b4dfbd..06384280ef4010e5767e8c7b2c087ba57774ab36 100644
--- a/chrome/version.gni
+++ b/chrome/version.gni
@@ -40,6 +40,11 @@ template("process_version") {
branding_path = "//chrome/app/theme/chromium/BRANDING"
}
+ extra_args = []
cjhopman 2014/10/23 21:42:05 document this new parameter above.
cjhopman 2014/10/23 21:42:05 Also, I think my preference would be to expose a l
newt (away) 2014/10/23 22:02:28 I considered that, but it didn't really seem worth
cjhopman 2014/10/23 22:05:48 sgtm
newt (away) 2014/10/23 22:25:24 Done.
+ if (defined(invoker.extra_args)) {
+ extra_args = invoker.extra_args
+ }
+
inputs = [
version_path,
invoker.source,
@@ -53,8 +58,8 @@ template("process_version") {
"-f", rebase_path(version_path, root_build_dir),
"-f", rebase_path(branding_path, root_build_dir),
"-f", rebase_path(lastchange_path, root_build_dir),
- rebase_path(invoker.source, root_build_dir),
- rebase_path(invoker.output, root_build_dir),
- ]
+ "-i", rebase_path(invoker.source, root_build_dir),
+ "-o", rebase_path(invoker.output, root_build_dir),
+ ] + extra_args
}
}
« chrome/android/channel.gni ('K') | « chrome/chrome_android.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698