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

Unified Diff: tools/testing/dart/compiler_configuration.dart

Issue 1659163007: Rename fletch -> dartino (Closed) Base URL: https://github.com/dartino/sdk.git@master
Patch Set: address comments Created 4 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/test.dart ('k') | tools/testing/dart/dartino_session_command.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/compiler_configuration.dart
diff --git a/tools/testing/dart/compiler_configuration.dart b/tools/testing/dart/compiler_configuration.dart
index cae0903bfba831c18d19100bb38feeff51dc9ee5..02f3fcf5cbf8321719864c2c9a47bf0a26ab6d04 100644
--- a/tools/testing/dart/compiler_configuration.dart
+++ b/tools/testing/dart/compiler_configuration.dart
@@ -19,8 +19,8 @@ import 'test_suite.dart' show
TestInformation,
TestUtils;
-import 'fletch_session_command.dart' show
- FletchSessionCommand;
+import 'dartino_session_command.dart' show
+ DartinoSessionCommand;
/// Grouping of a command with its expected result.
class CommandArtifact {
@@ -65,8 +65,8 @@ abstract class CompilerConfiguration {
bool isCsp = configuration['csp'];
switch (compiler) {
- case 'fletchc':
- return new FletchCCompilerConfiguration(
+ case 'dartino_compiler':
+ return new DartinoCCompilerConfiguration(
isDebug: isDebug, isChecked: isChecked,
isHostChecked: isHostChecked, useSdk: useSdk);
case 'none':
@@ -170,8 +170,8 @@ class Dart2xCompilerConfiguration extends CompilerConfiguration {
isHostChecked: isHostChecked, useSdk: useSdk);
String computeCompilerPath(String buildDir) {
- return useSdk ? '$buildDir/dartino-sdk/bin/fletch-vm'
- : '$buildDir/fletch-vm';
+ return useSdk ? '$buildDir/dartino-sdk/bin/dartino-vm'
+ : '$buildDir/dartino-vm';
}
CompilationCommand computeCompilationCommand(
@@ -199,19 +199,19 @@ class Dart2xCompilerConfiguration extends CompilerConfiguration {
}
}
-class FletchCCompilerConfiguration extends Dart2xCompilerConfiguration {
+class DartinoCCompilerConfiguration extends Dart2xCompilerConfiguration {
- FletchCCompilerConfiguration({
+ DartinoCCompilerConfiguration({
bool isDebug,
bool isChecked,
bool isHostChecked,
bool useSdk})
: super(
- 'fletchc',
+ 'dartino_compiler',
isDebug: isDebug, isChecked: isChecked,
isHostChecked: isHostChecked, useSdk: useSdk) {
if (!isHostChecked) {
- throw "fletch only works with --host-checked option.";
+ throw "dartino only works with --host-checked option.";
}
}
@@ -221,21 +221,21 @@ class FletchCCompilerConfiguration extends Dart2xCompilerConfiguration {
CommandBuilder commandBuilder,
List basicArguments,
Map<String, String> environmentOverrides) {
- String snapshotFileName = '$tempDir/fletch.snapshot';
- String executable = useSdk ? '$buildDir/dartino-sdk/bin/fletch'
- : '$buildDir/fletch';
+ String snapshotFileName = '$tempDir/dartino.snapshot';
+ String executable = useSdk ? '$buildDir/dartino-sdk/bin/dartino'
+ : '$buildDir/dartino';
String dart = useSdk ? '$buildDir/dartino-sdk/internal/dart'
: '$buildDir/dart';
Map<String, String> environment = {
'DART_VM' : dart,
}..addAll(environmentOverrides);
- Command command = new FletchSessionCommand(
+ Command command = new DartinoSessionCommand(
executable, basicArguments.first, basicArguments,
environment, snapshotFileName: snapshotFileName);
return new CommandArtifact(
- <Command>[command], snapshotFileName, 'application/fletch-snapshot');
+ <Command>[command], snapshotFileName, 'application/dartino-snapshot');
}
List<String> computeRuntimeArguments(
« no previous file with comments | « tools/test.dart ('k') | tools/testing/dart/dartino_session_command.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698