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

Side by Side Diff: tools/testing/dart/compiler_configuration.dart

Issue 2516433004: Add wrapper shell script to invoke dartk with checked-in binary (Closed)
Patch Set: Created 4 years, 1 month 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 | « tools/dartk_wrappers/dartk ('k') | 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) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 library compiler_configuration; 5 library compiler_configuration;
6 6
7 import 'dart:io' show Platform; 7 import 'dart:io' show Platform;
8 8
9 import 'runtime_configuration.dart' show RuntimeConfiguration; 9 import 'runtime_configuration.dart' show RuntimeConfiguration;
10 10
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 } 210 }
211 } 211 }
212 212
213 /// The "dartk" compiler. 213 /// The "dartk" compiler.
214 class DartKCompilerConfiguration extends CompilerConfiguration { 214 class DartKCompilerConfiguration extends CompilerConfiguration {
215 DartKCompilerConfiguration({bool isHostChecked, bool useSdk}) 215 DartKCompilerConfiguration({bool isHostChecked, bool useSdk})
216 : super._subclass(isHostChecked: isHostChecked, useSdk: useSdk); 216 : super._subclass(isHostChecked: isHostChecked, useSdk: useSdk);
217 217
218 @override 218 @override
219 String computeCompilerPath(String buildDir) { 219 String computeCompilerPath(String buildDir) {
220 return 'third_party/pkg/kernel/bin/dartk.dart'; 220 return 'tools/dartk_wrappers/dartk$executableScriptSuffix';
kustermann 2016/11/18 16:32:39 At some point we will add a tools/dartk_wrappers/d
221 } 221 }
222 222
223 CompilationCommand computeCompilationCommand( 223 CompilationCommand computeCompilationCommand(
224 String outputFileName, 224 String outputFileName,
225 String buildDir, 225 String buildDir,
226 CommandBuilder commandBuilder, 226 CommandBuilder commandBuilder,
227 List arguments, 227 List arguments,
228 Map<String, String> environmentOverrides) { 228 Map<String, String> environmentOverrides) {
229 var extraArguments = [ 229 var extraArguments = [
230 '--sdk', 230 '--sdk',
(...skipping 597 matching lines...) Expand 10 before | Expand all | Expand 10 after
828 RuntimeConfiguration runtimeConfiguration, 828 RuntimeConfiguration runtimeConfiguration,
829 String buildDir, 829 String buildDir,
830 TestInformation info, 830 TestInformation info,
831 List<String> vmOptions, 831 List<String> vmOptions,
832 List<String> sharedOptions, 832 List<String> sharedOptions,
833 List<String> originalArguments, 833 List<String> originalArguments,
834 CommandArtifact artifact) { 834 CommandArtifact artifact) {
835 return <String>[]; 835 return <String>[];
836 } 836 }
837 } 837 }
OLDNEW
« no previous file with comments | « tools/dartk_wrappers/dartk ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698