Commit cc72a990 authored by garykac's avatar garykac Committed by Commit bot

[Chromoting] Set ar_service_environment for Official builds

There is a check in build-webapp.py (line 197) that ensures that
non-dev builds have this set to 'prod'. Up 'til now, we've been
setting this manually (hence the check in build-webapp), but it
is better to set it correctly in the gyp file.

This also allows the bots to build Official builds without
triggering the build-webapp check.

BUG=

Review URL: https://codereview.chromium.org/886543002

Cr-Commit-Position: refs/heads/master@{#313558}
parent 230c8456
...@@ -15,9 +15,17 @@ ...@@ -15,9 +15,17 @@
'run_jscompile%': 0, 'run_jscompile%': 0,
# This variable is used to define the target environment for the app # The ar_service_environment variable is used to define the target
# being built. The allowed values are dev, test, staging, and prod. # environment for the app being built.
'ar_service_environment%': 'dev', # 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 }, # end of variables
'target_defaults': { 'target_defaults': {
...@@ -90,7 +98,7 @@ ...@@ -90,7 +98,7 @@
'<@(ar_generated_html_files)', '<@(ar_generated_html_files)',
'<(ar_app_manifest_app)', '<(ar_app_manifest_app)',
'<(DEPTH)/remoting/<(ar_app_manifest_common)', '<(DEPTH)/remoting/<(ar_app_manifest_common)',
], ],
'outputs': [ 'outputs': [
'<(output_dir)', '<(output_dir)',
'<(zip_path)', '<(zip_path)',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment