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

Side by Side Diff: dart.gyp

Issue 13639009: Make the create_sdk build dependent on the dartanalyzer build. This is in (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 8 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 | Annotate | Revision Log
« 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 (c) 2012, the Dart project authors. Please see the AUTHORS file 1 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 # for details. All rights reserved. Use of this source code is governed by a 2 # for details. All rights reserved. Use of this source code is governed by a
3 # BSD-style license that can be found in the LICENSE file. 3 # BSD-style license that can be found in the LICENSE file.
4 4
5 { 5 {
6 'targets': [ 6 'targets': [
7 { 7 {
8 # Contains all dependencies. 8 # Contains all dependencies.
9 'target_name': 'everything', 9 'target_name': 'everything',
10 'type': 'none', 10 'type': 'none',
(...skipping 13 matching lines...) Expand all
24 'type': 'none', 24 'type': 'none',
25 'dependencies': [ 25 'dependencies': [
26 'analyzer', 26 'analyzer',
27 'compiler', 27 'compiler',
28 'create_sdk', 28 'create_sdk',
29 'dart2js', 29 'dart2js',
30 'editor', 30 'editor',
31 'packages', 31 'packages',
32 'runtime', 32 'runtime',
33 'samples', 33 'samples',
34 'upload_sdk',
35 ], 34 ],
36 }, 35 },
37 { 36 {
38 'target_name': 'compiler', 37 'target_name': 'compiler',
39 'type': 'none', 38 'type': 'none',
40 'dependencies': [ 39 'dependencies': [
41 'compiler/dart-compiler.gyp:dart_analyzer', 40 'compiler/dart-compiler.gyp:dart_analyzer',
42 ], 41 ],
43 'actions': [] 42 'actions': []
44 }, 43 },
(...skipping 13 matching lines...) Expand all
58 ], 57 ],
59 }, 58 },
60 { 59 {
61 # Build the SDK. This target is separate from upload_sdk as the 60 # Build the SDK. This target is separate from upload_sdk as the
62 # editor needs to build the SDK without uploading it. 61 # editor needs to build the SDK without uploading it.
63 'target_name': 'create_sdk', 62 'target_name': 'create_sdk',
64 'type': 'none', 63 'type': 'none',
65 'dependencies': [ 64 'dependencies': [
66 'runtime/dart-runtime.gyp:dart', 65 'runtime/dart-runtime.gyp:dart',
67 'utils/compiler/compiler.gyp:dart2js', 66 'utils/compiler/compiler.gyp:dart2js',
67 'analyzer',
68 'compiler', 68 'compiler',
ahe 2013/04/08 08:08:56 Shouldn't you remove the old analyzer?
devoncarew 2013/04/08 09:56:58 Our current plan is to have a few weeks of overlap
69 'packages' 69 'packages'
70 ], 70 ],
71 'actions': [ 71 'actions': [
72 { 72 {
73 'action_name': 'create_sdk_py', 73 'action_name': 'create_sdk_py',
74 'inputs': [ 74 'inputs': [
75 '<!@(["python", "tools/list_files.py", "\\.dart$", "sdk/lib"])', 75 '<!@(["python", "tools/list_files.py", "\\.dart$", "sdk/lib"])',
76 '<!@(["python", "tools/list_files.py", "", "sdk/bin"])', 76 '<!@(["python", "tools/list_files.py", "", "sdk/bin"])',
77 'tools/create_sdk.py', 77 'tools/create_sdk.py',
78 '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)dart<(EXECUTABLE_SUFFIX)', 78 '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)dart<(EXECUTABLE_SUFFIX)',
79 '<(PRODUCT_DIR)/dart2js.snapshot', 79 '<(PRODUCT_DIR)/dart2js.snapshot',
80 '<(PRODUCT_DIR)/analyzer/bin/dart_analyzer', 80 '<(PRODUCT_DIR)/analyzer/bin/dart_analyzer',
81 '<(PRODUCT_DIR)/dartanalyzer/dartanalyzer.jar',
81 ], 82 ],
82 'outputs': [ 83 'outputs': [
83 '<(PRODUCT_DIR)/dart-sdk/README', 84 '<(PRODUCT_DIR)/dart-sdk/README',
84 ], 85 ],
85 'action': [ 86 'action': [
86 'python', 87 'python',
87 'tools/create_sdk.py', 88 'tools/create_sdk.py',
88 '<(PRODUCT_DIR)/dart-sdk', 89 '<(PRODUCT_DIR)/dart-sdk',
89 ], 90 ],
90 'message': 'Creating SDK.', 91 'message': 'Creating SDK.',
91 }, 92 },
92 ], 93 ],
93 }, 94 },
94 { 95 {
95 # Upload the SDK. This target is separate from create_sdk as the
96 # editor needs to build the SDK without uploading it.
97 'target_name': 'upload_sdk',
98 'type': 'none',
99 'dependencies': [
100 'create_sdk',
101 ],
102 'actions': [
103 {
104 'action_name': 'upload_sdk_py',
105 'inputs': [
106 '<(PRODUCT_DIR)/dart-sdk/README',
107 'tools/upload_sdk.py',
ahe 2013/04/08 08:08:56 Should you remove this script as well?
devoncarew 2013/04/08 09:56:58 I'd like to hold off on that and see what Dan says
108 ],
109 'outputs': [
110 '<(PRODUCT_DIR)/dart-sdk/upload.stamp',
111 ],
112 'action': [
113 'python',
114 'tools/upload_sdk.py',
115 '<(PRODUCT_DIR)/dart-sdk'
116 ],
117 },
118 ],
119 },
120 {
121 'target_name': 'dart2js', 96 'target_name': 'dart2js',
122 'type': 'none', 97 'type': 'none',
123 'dependencies': [ 98 'dependencies': [
124 'third_party/v8/src/d8.gyp:d8', 99 'third_party/v8/src/d8.gyp:d8',
125 'utils/compiler/compiler.gyp:dart2js', 100 'utils/compiler/compiler.gyp:dart2js',
126 ], 101 ],
127 }, 102 },
128 { 103 {
129 'target_name': 'analyzer', 104 'target_name': 'analyzer',
130 'type': 'none', 105 'type': 'none',
131 'dependencies': [ 106 'dependencies': [
132 'create_sdk',
133 'editor/analyzer.gyp:analyzer', 107 'editor/analyzer.gyp:analyzer',
134 ], 108 ],
135 }, 109 },
136 { 110 {
137 # This is the target that is built on the dart2dart bots. 111 # This is the target that is built on the dart2dart bots.
138 # It must depend on anything that is required by dart2dart 112 # It must depend on anything that is required by dart2dart
139 # tests. 113 # tests.
140 'target_name': 'dart2dart_bot', 114 'target_name': 'dart2dart_bot',
141 'type': 'none', 115 'type': 'none',
142 'dependencies': [ 116 'dependencies': [
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
229 }, 203 },
230 { 204 {
231 'target_name': 'packages', 205 'target_name': 'packages',
232 'type': 'none', 206 'type': 'none',
233 'dependencies': [ 207 'dependencies': [
234 'pkg/pkg.gyp:pkg_packages', 208 'pkg/pkg.gyp:pkg_packages',
235 ], 209 ],
236 }, 210 },
237 ], 211 ],
238 } 212 }
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