Skip to content

Commit

Permalink
Reverts "Manual Dart roll to 0ac840ba1 and update frontend server sna…
Browse files Browse the repository at this point in the history
…pshot filename (#51921)" (#51927)

Reverts: #51921
Initiated by: zanderso
Reason for reverting: This Dart roll is blocking the roll of the engine to the framework. Unblocking the rolls depends on addressing flutter/flutter#146164.
Original PR Author: jason-simmons

Reviewed By: {zanderso, jonahwilliams}

This change reverts the following previous change:
The Dart SDK is now only building an AOT snapshot for the frontend server (see https://dart-review.googlesource.com/c/sdk/+/359100)
  • Loading branch information
auto-submit[bot] committed Apr 5, 2024
1 parent c281924 commit f17d586
Show file tree
Hide file tree
Showing 8 changed files with 16 additions and 18 deletions.
2 changes: 1 addition & 1 deletion DEPS
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ vars = {
# Dart is: https://github.com/dart-lang/sdk/blob/main/DEPS
# You can use //tools/dart/create_updated_flutter_deps.py to produce
# updated revision list of existing dependencies.
'dart_revision': '0ac840ba1f0bf840f9f3e4859003e68324f3fd3b',
'dart_revision': '5b812b9ec2204cf6ae1a9e8204faa22b69ce8ace',

# WARNING: DO NOT EDIT MANUALLY
# The lines between blank lines above and below are generated by a script. See create_updated_flutter_deps.py
Expand Down
4 changes: 2 additions & 2 deletions build/archives/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -108,8 +108,8 @@ if (build_engine_artifacts && !is_android) {
destination = "vm_isolate_snapshot.bin"
},
{
source = "$root_gen_dir/frontend_server_aot.dart.snapshot"
destination = "frontend_server_aot.dart.snapshot"
source = "$root_gen_dir/frontend_server.dart.snapshot"
destination = "frontend_server.dart.snapshot"
},
]
}
Expand Down
5 changes: 2 additions & 3 deletions build/dart/rules.gni
Original file line number Diff line number Diff line change
Expand Up @@ -94,10 +94,9 @@ template("flutter_frontend_server") {
if (is_win) {
ext = ".exe"
}
dart = rebase_path("$host_prebuilt_dart_sdk/bin/dartaotruntime$ext",
root_out_dir)
dart = rebase_path("$host_prebuilt_dart_sdk/bin/dart$ext", root_out_dir)
frontend_server =
rebase_path("$root_gen_dir/frontend_server_aot.dart.snapshot")
rebase_path("$root_gen_dir/frontend_server.dart.snapshot")

args = [ dart ] + common_vm_args + [ frontend_server ] + common_args
}
Expand Down
9 changes: 5 additions & 4 deletions flutter_frontend_server/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,13 @@ import("//flutter/common/config.gni")

copy("frontend_server") {
if (flutter_prebuilt_dart_sdk) {
snapshot = "$host_prebuilt_dart_sdk/bin/snapshots/frontend_server_aot.dart.snapshot"
snapshot =
"$host_prebuilt_dart_sdk/bin/snapshots/frontend_server.dart.snapshot"
} else {
deps = [ "$dart_src/utils/kernel-service:frontend_server_aot" ]
snapshot = "$root_out_dir/frontend_server_aot.dart.snapshot"
deps = [ "$dart_src/utils/kernel-service:frontend_server" ]
snapshot = "$root_out_dir/frontend_server.dart.snapshot"
}

sources = [ snapshot ]
outputs = [ "$root_gen_dir/frontend_server_aot.dart.snapshot" ]
outputs = [ "$root_gen_dir/frontend_server.dart.snapshot" ]
}
5 changes: 2 additions & 3 deletions flutter_frontend_server/test/to_string_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ Future<void> main(List<String> args) async {
}

final String dart = Platform.resolvedExecutable;
final String dartaotruntime = path.join(path.dirname(Platform.resolvedExecutable), 'dartaotruntime');
final String buildDir = args[0];
final String frontendServer = args[1];
final String sdkRoot = args[2];
Expand All @@ -36,7 +35,7 @@ Future<void> main(List<String> args) async {
}

test('Without flag', () {
checkProcessResult(Process.runSync(dartaotruntime, <String>[
checkProcessResult(Process.runSync(dart, <String>[
frontendServer,
'--sdk-root=$sdkRoot',
'--target=flutter',
Expand All @@ -60,7 +59,7 @@ Future<void> main(List<String> args) async {
});

test('With flag', () {
checkProcessResult(Process.runSync(dartaotruntime, <String>[
checkProcessResult(Process.runSync(dart, <String>[
frontendServer,
'--sdk-root=$sdkRoot',
'--target=flutter',
Expand Down
2 changes: 1 addition & 1 deletion shell/platform/fuchsia/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ fuchsia_host_bundle("dart_binaries") {

_frontend_server_path =
rebase_path(get_label_info(_frontend_server_label, "root_gen_dir") +
"/frontend_server_aot.dart.snapshot")
"/frontend_server.dart.snapshot")

_list_libraries_path =
rebase_path(get_label_info(_list_libraries_label, "root_gen_dir") +
Expand Down
4 changes: 2 additions & 2 deletions testing/run_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -932,7 +932,7 @@ def build_dart_host_test_list(build_dir):
os.path.join('flutter', 'flutter_frontend_server'),
[
build_dir,
os.path.join(build_dir, 'gen', 'frontend_server_aot.dart.snapshot'),
os.path.join(build_dir, 'gen', 'frontend_server.dart.snapshot'),
os.path.join(build_dir, 'flutter_patched_sdk')
],
),
Expand All @@ -949,7 +949,7 @@ def build_dart_host_test_list(build_dir):
(
os.path.join('flutter', 'tools', 'const_finder'),
[
os.path.join(build_dir, 'gen', 'frontend_server_aot.dart.snapshot'),
os.path.join(build_dir, 'gen', 'frontend_server.dart.snapshot'),
os.path.join(build_dir, 'flutter_patched_sdk'),
os.path.join(build_dir, 'dart-sdk', 'lib', 'libraries.json'),
],
Expand Down
3 changes: 1 addition & 2 deletions tools/const_finder/test/const_finder_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ void expectInstances(dynamic value, dynamic expected, Compiler compiler) {
// This test is assuming the `dart` used to invoke the tests is compatible
// with the version of package:kernel in //third-party/dart/pkg/kernel
final String dart = Platform.resolvedExecutable;
final String dartaotruntime = path.join(path.dirname(Platform.resolvedExecutable), 'dartaotruntime');

void _checkRecursion(String dillPath, Compiler compiler) {
stdout.writeln('Checking recursive calls.');
Expand Down Expand Up @@ -444,7 +443,7 @@ class _Test {
}

void _compileAOTDill() {
checkProcessResult(Process.runSync(dartaotruntime, <String>[
checkProcessResult(Process.runSync(dart, <String>[
frontendServer,
'--sdk-root=$sdkRoot',
'--target=flutter',
Expand Down

0 comments on commit f17d586

Please sign in to comment.