[flutter_tool] In 'attach' use platform dill and patched sdk dir from the Fuchsia SDK (#32071)
diff --git a/packages/flutter_tools/lib/src/commands/attach.dart b/packages/flutter_tools/lib/src/commands/attach.dart
index 6a3e9d5..c822fcd 100644
--- a/packages/flutter_tools/lib/src/commands/attach.dart
+++ b/packages/flutter_tools/lib/src/commands/attach.dart
@@ -6,7 +6,9 @@
import 'package:multicast_dns/multicast_dns.dart';
+import '../artifacts.dart';
import '../base/common.dart';
+import '../base/context.dart';
import '../base/file_system.dart';
import '../base/io.dart';
import '../base/utils.dart';
@@ -160,8 +162,6 @@
@override
Future<FlutterCommandResult> runCommand() async {
- final FlutterProject flutterProject = FlutterProject.current();
-
Cache.releaseLockEarly();
await _validateArguments();
@@ -169,6 +169,19 @@
writePidFile(argResults['pid-file']);
final Device device = await findTargetDevice();
+
+ final Artifacts artifacts = device.artifactOverrides ?? Artifacts.instance;
+ await context.run<void>(
+ body: () => _attachToDevice(device),
+ overrides: <Type, Generator>{
+ Artifacts: () => artifacts,
+ });
+
+ return null;
+ }
+
+ Future<void> _attachToDevice(Device device) async {
+ final FlutterProject flutterProject = FlutterProject.current();
Future<int> getDevicePort() async {
if (debugPort != null) {
return debugPort;
@@ -310,7 +323,6 @@
await device.portForwarder.unforward(port);
}
}
- return null;
}
Future<void> _validateArguments() async { }
diff --git a/packages/flutter_tools/lib/src/device.dart b/packages/flutter_tools/lib/src/device.dart
index 8765e54..fa0ae74 100644
--- a/packages/flutter_tools/lib/src/device.dart
+++ b/packages/flutter_tools/lib/src/device.dart
@@ -7,6 +7,7 @@
import 'android/android_device.dart';
import 'application_package.dart';
+import 'artifacts.dart';
import 'base/context.dart';
import 'base/file_system.dart';
import 'base/utils.dart';
@@ -274,6 +275,9 @@
/// Clear the device's logs.
void clearLogs();
+ /// Optional device-specific artifact overrides.
+ OverrideArtifacts get artifactOverrides => null;
+
/// Start an app package on the current device.
///
/// [platformArgs] allows callers to pass platform-specific arguments to the
diff --git a/packages/flutter_tools/lib/src/fuchsia/fuchsia_device.dart b/packages/flutter_tools/lib/src/fuchsia/fuchsia_device.dart
index a2fddc2..f9fa27b 100644
--- a/packages/flutter_tools/lib/src/fuchsia/fuchsia_device.dart
+++ b/packages/flutter_tools/lib/src/fuchsia/fuchsia_device.dart
@@ -7,6 +7,7 @@
import 'package:meta/meta.dart';
import '../application_package.dart';
+import '../artifacts.dart';
import '../base/common.dart';
import '../base/io.dart';
import '../base/logger.dart';
@@ -224,6 +225,16 @@
void clearLogs() {}
@override
+ OverrideArtifacts get artifactOverrides {
+ return _artifactOverrides ??= OverrideArtifacts(
+ parent: Artifacts.instance,
+ platformKernelDill: fuchsiaArtifacts.platformKernelDill,
+ flutterPatchedSdk: fuchsiaArtifacts.flutterPatchedSdk,
+ );
+ }
+ OverrideArtifacts _artifactOverrides;
+
+ @override
bool get supportsScreenshot => false;
bool get ipv6 {
diff --git a/packages/flutter_tools/lib/src/fuchsia/fuchsia_sdk.dart b/packages/flutter_tools/lib/src/fuchsia/fuchsia_sdk.dart
index b246a09..d7d3a51 100644
--- a/packages/flutter_tools/lib/src/fuchsia/fuchsia_sdk.dart
+++ b/packages/flutter_tools/lib/src/fuchsia/fuchsia_sdk.dart
@@ -81,7 +81,12 @@
/// Fuchsia-specific artifacts used to interact with a device.
class FuchsiaArtifacts {
/// Creates a new [FuchsiaArtifacts].
- FuchsiaArtifacts({this.sshConfig, this.devFinder});
+ FuchsiaArtifacts({
+ this.sshConfig,
+ this.devFinder,
+ this.platformKernelDill,
+ this.flutterPatchedSdk,
+ });
/// Creates a new [FuchsiaArtifacts] using the cached Fuchsia SDK.
///
@@ -92,6 +97,7 @@
factory FuchsiaArtifacts.find() {
final String fuchsia = Cache.instance.getArtifactDirectory('fuchsia').path;
final String tools = fs.path.join(fuchsia, 'tools');
+ final String dartPrebuilts = fs.path.join(tools, 'dart_prebuilts');
// If FUCHSIA_BUILD_DIR is defined, then look for the ssh_config dir
// relative to it. Next, if FUCHSIA_SSH_CONFIG is defined, then use it.
@@ -99,13 +105,17 @@
File sshConfig;
if (platform.environment.containsKey(_kFuchsiaBuildDir)) {
sshConfig = fs.file(fs.path.join(
- platform.environment[_kFuchsiaSshConfig], 'ssh-keys', 'ssh_config'));
+ platform.environment[_kFuchsiaBuildDir], 'ssh-keys', 'ssh_config'));
} else if (platform.environment.containsKey(_kFuchsiaSshConfig)) {
sshConfig = fs.file(platform.environment[_kFuchsiaSshConfig]);
}
return FuchsiaArtifacts(
sshConfig: sshConfig,
devFinder: fs.file(fs.path.join(tools, 'dev_finder')),
+ platformKernelDill: fs.file(fs.path.join(
+ dartPrebuilts, 'flutter_runner', 'platform_strong.dill')),
+ flutterPatchedSdk: fs.file(fs.path.join(
+ dartPrebuilts, 'flutter_runner')),
);
}
@@ -119,4 +129,10 @@
/// The location of the dev finder tool used to locate connected
/// Fuchsia devices.
final File devFinder;
+
+ /// The location of the Fuchsia-specific platform dill.
+ final File platformKernelDill;
+
+ /// The directory containing [platformKernelDill].
+ final File flutterPatchedSdk;
}
