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

Side by Side Diff: utils/compiler/build_helper.dart

Issue 10392023: Change dart:io to use Future for one-shot operations. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Adding stable test binaries Created 8 years, 7 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 | « tools/testing/dart/test_suite.dart ('k') | utils/pub/io.dart » ('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 (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 #import('dart:io'); 5 #import('dart:io');
6 #import('dart:uri'); 6 #import('dart:uri');
7 7
8 #import('../../lib/compiler/implementation/util/uri_extras.dart'); 8 #import('../../lib/compiler/implementation/util/uri_extras.dart');
9 #import('../../lib/compiler/implementation/filenames.dart'); 9 #import('../../lib/compiler/implementation/filenames.dart');
10 10
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 44
45 f = new File('${uriPathToNative(uri.path)}.bat'); 45 f = new File('${uriPathToNative(uri.path)}.bat');
46 stream = f.openSync(FileMode.WRITE); 46 stream = f.openSync(FileMode.WRITE);
47 try { 47 try {
48 stream.writeStringSync(batFile); 48 stream.writeStringSync(batFile);
49 } finally { 49 } finally {
50 stream.closeSync(); 50 stream.closeSync();
51 } 51 }
52 52
53 if (Platform.operatingSystem != 'windows') { 53 if (Platform.operatingSystem != 'windows') {
54 onExit(int exitCode, String stdout, String stderr) { 54 onExit(ProcessResult result) {
55 if (exitCode != 0) { 55 if (result.exitCode != 0) {
56 print(stdout); 56 print(result.stdout);
57 print(stderr); 57 print(result.stderr);
58 exit(exitCode); 58 exit(result.exitCode);
59 } 59 }
60 } 60 }
61 new Process.run('/bin/chmod', ['+x', uri.path], null, onExit); 61 Process.run('/bin/chmod', ['+x', uri.path]).then(onExit);
62 } 62 }
63 } 63 }
64 64
65 List<String> buildScript(Uri dartUri, Uri dartVmLocation, String options) { 65 List<String> buildScript(Uri dartUri, Uri dartVmLocation, String options) {
66 Uri dart2jsUri = dartUri.resolve('lib/compiler/implementation/dart2js.dart'); 66 Uri dart2jsUri = dartUri.resolve('lib/compiler/implementation/dart2js.dart');
67 String dart2jsPath = relativize(dartVmLocation, dart2jsUri); 67 String dart2jsPath = relativize(dartVmLocation, dart2jsUri);
68 String dart2jsPathWin = dart2jsPath.replaceAll("/", "\\"); 68 String dart2jsPathWin = dart2jsPath.replaceAll("/", "\\");
69 69
70 print('dartUri = $dartUri'); 70 print('dartUri = $dartUri');
71 print('dartVmLocation = $dartVmLocation'); 71 print('dartVmLocation = $dartVmLocation');
(...skipping 20 matching lines...) Expand all
92 set SCRIPTPATH=%~dp0 92 set SCRIPTPATH=%~dp0
93 93
94 REM Does the path have a trailing slash? If so, remove it. 94 REM Does the path have a trailing slash? If so, remove it.
95 if %SCRIPTPATH:~-1%==\ set SCRIPTPATH=%SCRIPTPATH:~0,-1% 95 if %SCRIPTPATH:~-1%==\ set SCRIPTPATH=%SCRIPTPATH:~0,-1%
96 96
97 set arguments=%* 97 set arguments=%*
98 98
99 "%SCRIPTPATH%\dart.exe"$options "%SCRIPTPATH%$dart2jsPathWin" %arguments% 99 "%SCRIPTPATH%\dart.exe"$options "%SCRIPTPATH%$dart2jsPathWin" %arguments%
100 '''.replaceAll('\n', '\r\n')]; 100 '''.replaceAll('\n', '\r\n')];
101 } 101 }
OLDNEW
« no previous file with comments | « tools/testing/dart/test_suite.dart ('k') | utils/pub/io.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698