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

Side by Side Diff: remoting/remoting_webapp.gypi

Issue 905553002: [Chromoting] Pass list of locale files into build-webapp in a single file. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove --locales option from build-webapp.py 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 | « remoting/app_remoting_webapp_build.gypi ('k') | remoting/tools/build/remoting_localize.py » ('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 # File included in remoting_webapp_* targets in remoting_client.gypi 5 # File included in remoting_webapp_* targets in remoting_client.gypi
6 6
7 { 7 {
8 'type': 'none', 8 'type': 'none',
9 'variables': { 9 'variables': {
10 'extra_files%': [], 10 'extra_files%': [],
11 'generated_html_files': [ 11 'generated_html_files': [
12 '<(SHARED_INTERMEDIATE_DIR)/main.html', 12 '<(SHARED_INTERMEDIATE_DIR)/main.html',
13 '<(SHARED_INTERMEDIATE_DIR)/wcs_sandbox.html', 13 '<(SHARED_INTERMEDIATE_DIR)/wcs_sandbox.html',
14 '<(SHARED_INTERMEDIATE_DIR)/background.html', 14 '<(SHARED_INTERMEDIATE_DIR)/background.html',
15 ], 15 ],
16 'dr_webapp_locales_listfile': '<(SHARED_INTERMEDIATE_DIR)/>(_target_name)_lo cales.txt',
16 }, 17 },
17 'dependencies': [ 18 'dependencies': [
18 'remoting_resources', 19 'remoting_resources',
19 'remoting_webapp_html', 20 'remoting_webapp_html',
20 ], 21 ],
21 'conditions': [ 22 'conditions': [
22 ['run_jscompile != 0', { 23 ['run_jscompile != 0', {
23 'variables': { 24 'variables': {
24 'success_stamp': '<(PRODUCT_DIR)/<(_target_name)_jscompile.stamp', 25 'success_stamp': '<(PRODUCT_DIR)/<(_target_name)_jscompile.stamp',
25 }, 26 },
(...skipping 14 matching lines...) Expand all
40 '--success-stamp', '<(success_stamp)', 41 '--success-stamp', '<(success_stamp)',
41 '<@(remoting_webapp_crd_js_files)', 42 '<@(remoting_webapp_crd_js_files)',
42 '<@(remoting_webapp_js_proto_files)', 43 '<@(remoting_webapp_js_proto_files)',
43 ], 44 ],
44 }, 45 },
45 ], # actions 46 ], # actions
46 }], 47 }],
47 ], 48 ],
48 'actions': [ 49 'actions': [
49 { 50 {
51 'action_name': 'Build Remoting locales listfile',
52 'inputs': [
53 '<(remoting_localize_path)',
54 ],
55 'outputs': [
56 '<(dr_webapp_locales_listfile)',
57 ],
58 'action': [
59 'python', '<(remoting_localize_path)',
Sergey Ulanov 2015/02/09 23:53:15 You don't really need this action. GYP allows to w
60 '--locale_output',
61 '"<(webapp_locale_dir)/@{json_suffix}/messages.json"',
62 '--locales_listfile',
63 '<(dr_webapp_locales_listfile)',
64 '<@(remoting_locales)',
65 ],
66 },
67 {
50 'action_name': 'Build Remoting WebApp', 68 'action_name': 'Build Remoting WebApp',
51 'inputs': [ 69 'inputs': [
52 'webapp/build-webapp.py', 70 'webapp/build-webapp.py',
53 'webapp/crd/manifest.json.jinja2', 71 'webapp/crd/manifest.json.jinja2',
54 '<(chrome_version_path)', 72 '<(chrome_version_path)',
55 '<(remoting_version_path)', 73 '<(remoting_version_path)',
74 '<(dr_webapp_locales_listfile)',
56 '<@(generated_html_files)', 75 '<@(generated_html_files)',
57 '<@(remoting_webapp_crd_files)', 76 '<@(remoting_webapp_crd_files)',
58 '<@(remoting_webapp_locale_files)', 77 '<@(remoting_webapp_locale_files)',
59 '<@(extra_files)', 78 '<@(extra_files)',
60 ], 79 ],
61 'outputs': [ 80 'outputs': [
62 '<(output_dir)', 81 '<(output_dir)',
63 '<(zip_path)', 82 '<(zip_path)',
64 ], 83 ],
65 'action': [ 84 'action': [
66 'python', 'webapp/build-webapp.py', 85 'python', 'webapp/build-webapp.py',
67 '<(buildtype)', 86 '<(buildtype)',
68 '<(version_full)', 87 '<(version_full)',
69 '<(output_dir)', 88 '<(output_dir)',
70 '<(zip_path)', 89 '<(zip_path)',
71 'webapp/crd/manifest.json.jinja2', 90 'webapp/crd/manifest.json.jinja2',
72 '<(webapp_type)', 91 '<(webapp_type)',
73 '<@(generated_html_files)', 92 '<@(generated_html_files)',
74 '<@(remoting_webapp_crd_files)', 93 '<@(remoting_webapp_crd_files)',
75 '<@(extra_files)', 94 '<@(extra_files)',
76 '--locales', '<@(remoting_webapp_locale_files)', 95 '--locales_listfile',
96 '<(dr_webapp_locales_listfile)',
77 ], 97 ],
78 }, 98 },
79 ], 99 ],
80 } 100 }
OLDNEW
« no previous file with comments | « remoting/app_remoting_webapp_build.gypi ('k') | remoting/tools/build/remoting_localize.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698