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

Side by Side 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, 10 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'includes': [ 6 'includes': [
7 'remoting_version.gypi', 7 'remoting_version.gypi',
8 'remoting_locales.gypi', 8 'remoting_locales.gypi',
9 'remoting_webapp_files.gypi', 9 'remoting_webapp_files.gypi',
10 'app_remoting_webapp_files.gypi', 10 'app_remoting_webapp_files.gypi',
11 ], 11 ],
12 12
13 'variables': { 13 'variables': {
14 'chromium_code': 1, 14 'chromium_code': 1,
15 15
16 'run_jscompile%': 0, 16 'run_jscompile%': 0,
17 17
18 # This variable is used to define the target environment for the app 18 # The ar_service_environment variable is used to define the target
19 # being built. The allowed values are dev, test, staging, and prod. 19 # environment for the app being built.
20 'ar_service_environment%': 'dev', 20 # The allowed values are dev, test, staging, and prod.
21 'conditions': [
22 ['buildtype == "Dev"', {
23 'ar_service_environment%': 'dev',
24 }, { # buildtype != 'Dev'
25 # Non-dev build must have this set to 'prod'.
26 'ar_service_environment': 'prod',
27 }],
28 ], # conditions
21 }, # end of variables 29 }, # end of variables
22 30
23 'target_defaults': { 31 'target_defaults': {
24 'type': 'none', 32 'type': 'none',
25 33
26 'dependencies': [ 34 'dependencies': [
27 # TODO(wez): Create proper resources for shared-stub and app-specific 35 # TODO(wez): Create proper resources for shared-stub and app-specific
28 # stubs. 36 # stubs.
29 '<(DEPTH)/remoting/remoting.gyp:remoting_resources', 37 '<(DEPTH)/remoting/remoting.gyp:remoting_resources',
30 ], 38 ],
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 'action_name': 'Build ">(ar_app_name)" application stub', 91 'action_name': 'Build ">(ar_app_name)" application stub',
84 'inputs': [ 92 'inputs': [
85 '<(DEPTH)/remoting/webapp/build-webapp.py', 93 '<(DEPTH)/remoting/webapp/build-webapp.py',
86 '<(chrome_version_path)', 94 '<(chrome_version_path)',
87 '<(remoting_version_path)', 95 '<(remoting_version_path)',
88 '<@(ar_webapp_files)', 96 '<@(ar_webapp_files)',
89 '<@(remoting_webapp_locale_files)', 97 '<@(remoting_webapp_locale_files)',
90 '<@(ar_generated_html_files)', 98 '<@(ar_generated_html_files)',
91 '<(ar_app_manifest_app)', 99 '<(ar_app_manifest_app)',
92 '<(DEPTH)/remoting/<(ar_app_manifest_common)', 100 '<(DEPTH)/remoting/<(ar_app_manifest_common)',
93 ], 101 ],
94 'outputs': [ 102 'outputs': [
95 '<(output_dir)', 103 '<(output_dir)',
96 '<(zip_path)', 104 '<(zip_path)',
97 ], 105 ],
98 'action': [ 106 'action': [
99 'python', '<(DEPTH)/remoting/webapp/build-webapp.py', 107 'python', '<(DEPTH)/remoting/webapp/build-webapp.py',
100 '<(buildtype)', 108 '<(buildtype)',
101 '<(version_full)', 109 '<(version_full)',
102 '<(output_dir)', 110 '<(output_dir)',
103 '<(zip_path)', 111 '<(zip_path)',
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
169 # rather than using the extension id. 177 # rather than using the extension id.
170 # This is only done for Dev apps because the app-id for Release apps 178 # This is only done for Dev apps because the app-id for Release apps
171 # *must* match the extension id. 179 # *must* match the extension id.
172 'variables': { 180 'variables': {
173 'remoting_app_id': ['--appid', '>(_app_id)'], 181 'remoting_app_id': ['--appid', '>(_app_id)'],
174 }, 182 },
175 }], 183 }],
176 ], # conditions 184 ], # conditions
177 }, # target_defaults 185 }, # target_defaults
178 } 186 }
OLDNEW
« 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