Skip to content

Commit

Permalink
Defer changes in v3.3.0-wip to v4.0.0-wip
Browse files Browse the repository at this point in the history
  • Loading branch information
derekxu16 committed Dec 19, 2023
1 parent 94c0539 commit 7ae2041
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 15 deletions.
8 changes: 3 additions & 5 deletions frontend_server_client/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,12 @@
## 4.0.0-wip

- Update Dart SDK constraint to `^3.0.0`.
- Support changes in the SDK layout for Dart 3.0.
- By default, start the frontend server from the AOT snapshot shipped in the
Dart SDK.
- Throw an `ArgumentError` when `FrontendServerClient.start` is called with the
`frontendServerPath` argument omitted and the `debug` argument set to true.

## 3.3.0

- Update Dart SDK constraint to `>=3.0.0 <4.0.0`.
- Support changes in the SDK layout for Dart 3.0.
- Update `package:vm_service` constraint to `^14.0.0`.

## 3.2.0

Expand Down
16 changes: 8 additions & 8 deletions frontend_server_client/lib/src/frontend_server_client.dart
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,10 @@ class FrontendServerClient {
feServer = await Process.start(
Platform.resolvedExecutable,
<String>[
if (debug) '--observe',
frontendServerPath,
...commonArguments,
],
if (debug) '--observe',
frontendServerPath,
...commonArguments,
],
);
} else if (File(_feServerAotSnapshotPath).existsSync()) {
if (debug) {
Expand All @@ -123,10 +123,10 @@ class FrontendServerClient {
feServer = await Process.start(
Platform.resolvedExecutable,
<String>[
if (debug) '--observe',
_feServerAppJitSnapshotPath,
...commonArguments,
],
if (debug) '--observe',
_feServerAppJitSnapshotPath,
...commonArguments,
],
);
}
var feServerStdoutLines = StreamQueue(feServer.stdout
Expand Down
2 changes: 1 addition & 1 deletion frontend_server_client/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,4 +20,4 @@ dev_dependencies:
test: ^1.16.0
test_descriptor: ^2.0.0
test_process: ^2.0.0
vm_service: ^8.0.0
vm_service: ^14.0.0
2 changes: 1 addition & 1 deletion frontend_server_common/lib/src/frontend_server_client.dart
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,7 @@ class ResidentCompiler {

_logger.info(args.join(' '));
final workingDirectory = projectDirectory.toFilePath();
_server = await Process.start(sdkLayout.dartPath, args,
_server = await Process.start(sdkLayout.dartAotRuntimePath, args,
workingDirectory: workingDirectory);

var server = _server!;
Expand Down
7 changes: 7 additions & 0 deletions test_common/lib/test_sdk_layout.dart
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,11 @@ class TestSdkLayout {
'bin',
Platform.isWindows ? 'dart.exe' : 'dart',
),
dartAotRuntimePath: p.join(
sdkLayout.sdkDirectory,
'bin',
Platform.isWindows ? 'dartaotruntime.exe' : 'dartaotruntime',
),
frontendServerSnapshotPath: p.join(
sdkLayout.sdkDirectory,
'bin',
Expand Down Expand Up @@ -137,6 +142,7 @@ class TestSdkLayout {
final String stackTraceMapperPath;

final String dartPath;
final String dartAotRuntimePath;
final String frontendServerSnapshotPath;
final String dartdevcSnapshotPath;
final String kernelWorkerSnapshotPath;
Expand All @@ -155,6 +161,7 @@ class TestSdkLayout {
required this.requireJsPath,
required this.stackTraceMapperPath,
required this.dartPath,
required this.dartAotRuntimePath,
required this.frontendServerSnapshotPath,
required this.dartdevcSnapshotPath,
required this.kernelWorkerSnapshotPath,
Expand Down

0 comments on commit 7ae2041

Please sign in to comment.