From 7cd233e1f01f7e00575e157f51ba3aa4de50fdf2 Mon Sep 17 00:00:00 2001
From: Fernando Trigoso <ftrigoso@dropsource.com>
Date: Fri, 6 Oct 2023 20:07:23 -0400
Subject: [PATCH] tools: read dart version from flutter sdk dir (#131)

---
 tools/build_controller.dart  |  6 +++---
 tools/build_preview_api.dart | 14 +++++++-------
 tools/paths.dart             |  6 ++++++
 3 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/tools/build_controller.dart b/tools/build_controller.dart
index dbe269e..4da38d5 100644
--- a/tools/build_controller.dart
+++ b/tools/build_controller.dart
@@ -33,10 +33,10 @@ Building Monarch Controller using these arguments:
     out_controller_dir.deleteSync(recursive: true);
   out_controller_dir.createSync(recursive: true);
 
-  var flutterVersion = pub.Version.parse(get_flutter_version(flutter_sdk));
-  var flutterVersionWithDart3 = pub.Version(3, 8, 0, pre: '10.1.pre');
+  var dartVersion = pub.Version.parse(get_dart_version(flutter_sdk));
+  var dart3 = pub.Version(3, 0, 0);
 
-  var useGrpc310 = flutterVersion < flutterVersionWithDart3;
+  var useGrpc310 = dartVersion < dart3;
   if (useGrpc310) {
     print('Running `git apply grpc_310.patch`\n');
     utils.gitApplyPatch(repo_paths.controller, 'grpc_310.patch');
diff --git a/tools/build_preview_api.dart b/tools/build_preview_api.dart
index 6f28d95..51807d2 100644
--- a/tools/build_preview_api.dart
+++ b/tools/build_preview_api.dart
@@ -10,7 +10,8 @@ import 'utils_local.dart' as local_utils;
 /// - Path to the root of the Monarch repo
 /// - Path to the Flutter SDK to use
 /// - Path to the monarch_ui/{flutter_id} output directory
-void buildPreviewApi(String repo_root, String flutter_sdk, String out_ui_flutter_id) {
+void buildPreviewApi(
+    String repo_root, String flutter_sdk, String out_ui_flutter_id) {
   var repo_paths = RepoPaths(repo_root);
 
   print('''
@@ -32,11 +33,10 @@ Building Monarch Preview API using these arguments:
     out_preview_api_dir.deleteSync(recursive: true);
   out_preview_api_dir.createSync(recursive: true);
 
+  var dartVersion = pub.Version.parse(get_dart_version(flutter_sdk));
+  var dart3 = pub.Version(3, 0, 0);
 
-  var flutterVersion = pub.Version.parse(get_flutter_version(flutter_sdk));
-  var flutterVersionWithDart3 = pub.Version(3, 8, 0, pre: '10.1.pre');
-
-  var useGrpc310 = flutterVersion < flutterVersionWithDart3;
+  var useGrpc310 = dartVersion < dart3;
   if (useGrpc310) {
     print('Running `git apply grpc_310.patch`\n');
     utils.gitApplyPatch(repo_paths.preview_api, 'grpc_310.patch');
@@ -81,7 +81,8 @@ Building monarch preview_api flutter bundle. Will output to:
   }
 
   {
-    var icudtl_dat_ = icudtl_dat(flutter_sdk, local_utils.read_target_platform());
+    var icudtl_dat_ =
+        icudtl_dat(flutter_sdk, local_utils.read_target_platform());
 
     if (Platform.isWindows) {
       var result = Process.runSync(
@@ -109,4 +110,3 @@ Building monarch preview_api flutter bundle. Will output to:
 ===============================================================================
 ''');
 }
-
diff --git a/tools/paths.dart b/tools/paths.dart
index a358f2f..68051e2 100644
--- a/tools/paths.dart
+++ b/tools/paths.dart
@@ -114,3 +114,9 @@ String gen_seed_flutter_id(String platform_gen_seed, String flutter_sdk) =>
 
 String build_flutter_id(String build, String flutter_sdk) =>
     p.join(build, flutter_id(flutter_sdk));
+
+String get_dart_version(String flutter_sdk) {
+  var version = File(p.join(flutter_sdk, 'bin', 'cache', 'dart-sdk', 'version'))
+      .readAsStringSync();
+  return version.trim();
+}