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

Unified Diff: remoting/app_remoting_webapp_build.gypi

Issue 886543002: [Chromoting] Set ar_service_environment for Official builds (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move condition to top. Created 5 years, 11 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: remoting/app_remoting_webapp_build.gypi
diff --git a/remoting/app_remoting_webapp_build.gypi b/remoting/app_remoting_webapp_build.gypi
index cf7e2af2d498b2dc257d8ddebbd1eb0b326e0bf1..1c3e68cff92d687a757353381d7548ae9d8688bd 100644
--- a/remoting/app_remoting_webapp_build.gypi
+++ b/remoting/app_remoting_webapp_build.gypi
@@ -15,9 +15,17 @@
'run_jscompile%': 0,
- # This variable is used to define the target environment for the app
- # being built. The allowed values are dev, test, staging, and prod.
- 'ar_service_environment%': 'dev',
+ # The ar_service_environment variable is used to define the target
+ # environment for the app being built.
+ # The allowed values are dev, test, staging, and prod.
+ 'conditions': [
+ ['buildtype == "Dev"', {
+ 'ar_service_environment%': 'dev',
+ }, { # buildtype != 'Dev'
+ # Non-dev build must have this set to 'prod'.
+ 'ar_service_environment': 'prod',
+ }],
+ ], # conditions
}, # end of variables
'target_defaults': {
@@ -90,7 +98,7 @@
'<@(ar_generated_html_files)',
'<(ar_app_manifest_app)',
'<(DEPTH)/remoting/<(ar_app_manifest_common)',
- ],
+ ],
'outputs': [
'<(output_dir)',
'<(zip_path)',
« 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