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

Side by Side Diff: remoting/app_remoting_webapp_build.gypi

Issue 1031203004: To avoid the requirement of defining unnecessary gyp variables for (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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 | remoting/remoting_options.gypi » ('j') | 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_options.gypi', 9 'remoting_options.gypi',
10 'remoting_webapp_files.gypi', 10 'remoting_webapp_files.gypi',
11 'app_remoting_webapp_files.gypi', 11 'app_remoting_webapp_files.gypi',
12 ], 12 ],
13 13
14 'variables': {
15 # The ar_service_environment variable is used to define the target
16 # environment for the app being built.
17 # The allowed values are dev, test, staging, and prod.
18 'conditions': [
19 ['buildtype == "Dev"', {
20 'ar_service_environment%': 'dev',
21 }, { # buildtype != 'Dev'
22 # Non-dev build must have this set to 'prod'.
23 'ar_service_environment': 'prod',
24 }],
25 ], # conditions
26 }, # end of variables
27
28 'target_defaults': { 14 'target_defaults': {
29 'type': 'none', 15 'type': 'none',
30 16
31 'dependencies': [ 17 'dependencies': [
32 # TODO(wez): Create proper resources for shared-stub and app-specific 18 # TODO(wez): Create proper resources for shared-stub and app-specific
33 # stubs. 19 # stubs.
34 '<(DEPTH)/remoting/remoting.gyp:remoting_resources', 20 '<(DEPTH)/remoting/remoting.gyp:remoting_resources',
35 ], 21 ],
36 22
37 'locale_files': [ 23 'locale_files': [
(...skipping 193 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 # rather than using the extension id. 217 # rather than using the extension id.
232 # This is only done for Dev apps because the app-id for Release apps 218 # This is only done for Dev apps because the app-id for Release apps
233 # *must* match the extension id. 219 # *must* match the extension id.
234 'variables': { 220 'variables': {
235 'remoting_app_id': ['--appid', '>(_app_id)'], 221 'remoting_app_id': ['--appid', '>(_app_id)'],
236 }, 222 },
237 }], 223 }],
238 ], # conditions 224 ], # conditions
239 }, # target_defaults 225 }, # target_defaults
240 } 226 }
OLDNEW
« no previous file with comments | « no previous file | remoting/remoting_options.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698