From 5b6fce7cb9964d9a46ceeeb4d642f645642f3aa3 Mon Sep 17 00:00:00 2001 From: Victoria Ashworth <15619084+vashworth@users.noreply.github.com> Date: Thu, 9 Nov 2023 11:38:49 -0600 Subject: [PATCH] Use specific version of mac_toolchain (flutter/engine#47835) Newer versions of mac_toolchain increase Xcode install times by ~2 minutes (https://github.com/flutter/flutter/issues/138109). As a temporary solution, we're using an older version of mac_toolchain. [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style --- engine/src/flutter/.ci.yaml | 15 +++++-- .../ci/builders/mac_android_aot_engine.json | 24 ++++++++--- .../flutter/ci/builders/mac_host_engine.json | 24 ++++++++--- .../flutter/ci/builders/mac_ios_engine.json | 40 ++++++++++++++----- engine/src/flutter/ci/builders/mac_unopt.json | 20 +++++++--- 5 files changed, 93 insertions(+), 30 deletions(-) diff --git a/engine/src/flutter/.ci.yaml b/engine/src/flutter/.ci.yaml index b6b83d5019..0f7e2a0976 100644 --- a/engine/src/flutter/.ci.yaml +++ b/engine/src/flutter/.ci.yaml @@ -54,9 +54,12 @@ platform_properties: device_type: none cpu: x86 os: Mac-12|Mac-13 + # TODO(vashworth): Remove specific toolchain_ver once https://github.com/flutter/flutter/issues/138109 is resolved. $flutter/osx_sdk : >- { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } dimensions: mac_model: "Macmini8,1|Macmini9,1" @@ -447,9 +450,12 @@ targets: [ {"dependency": "goldctl", "version": "git_revision:720a542f6fe4f92922c3b8f0fdcc4d2ac6bb83cd"} ] + # TODO(vashworth): Remove specific toolchain_ver once https://github.com/flutter/flutter/issues/138109 is resolved. $flutter/osx_sdk : >- { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } drone_dimensions: - os=Mac-12|Mac-13 @@ -468,9 +474,12 @@ targets: add_recipes_cq: "true" release_build: "true" config_name: mac_ios_engine + # TODO(vashworth): Remove specific toolchain_ver once https://github.com/flutter/flutter/issues/138109 is resolved. $flutter/osx_sdk : >- { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } drone_dimensions: - os=Mac-12|Mac-13 diff --git a/engine/src/flutter/ci/builders/mac_android_aot_engine.json b/engine/src/flutter/ci/builders/mac_android_aot_engine.json index 962250d0de..5704a2beb5 100644 --- a/engine/src/flutter/ci/builders/mac_android_aot_engine.json +++ b/engine/src/flutter/ci/builders/mac_android_aot_engine.json @@ -35,7 +35,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -75,7 +77,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -115,7 +119,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -154,7 +160,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -194,7 +202,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -234,7 +244,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } } diff --git a/engine/src/flutter/ci/builders/mac_host_engine.json b/engine/src/flutter/ci/builders/mac_host_engine.json index 8642c1b3e8..c00f8a5158 100644 --- a/engine/src/flutter/ci/builders/mac_host_engine.json +++ b/engine/src/flutter/ci/builders/mac_host_engine.json @@ -47,7 +47,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } }, "tests": [ @@ -106,7 +108,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } }, "tests": [ @@ -176,7 +180,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } }, "tests": [ @@ -237,7 +243,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -280,7 +288,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -325,7 +335,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } } diff --git a/engine/src/flutter/ci/builders/mac_ios_engine.json b/engine/src/flutter/ci/builders/mac_ios_engine.json index 08cc3adf97..67d43aaccc 100644 --- a/engine/src/flutter/ci/builders/mac_ios_engine.json +++ b/engine/src/flutter/ci/builders/mac_ios_engine.json @@ -19,7 +19,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -43,7 +45,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -67,7 +71,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -92,7 +98,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -117,7 +125,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -141,7 +151,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -166,7 +178,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -191,7 +205,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -217,7 +233,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -243,7 +261,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } } diff --git a/engine/src/flutter/ci/builders/mac_unopt.json b/engine/src/flutter/ci/builders/mac_unopt.json index ae72400e34..74aec4ed47 100644 --- a/engine/src/flutter/ci/builders/mac_unopt.json +++ b/engine/src/flutter/ci/builders/mac_unopt.json @@ -33,7 +33,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } }, "tests": [ @@ -66,7 +68,9 @@ "ios-16-4_14e300c", "ios-16-2_14c18" ], - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } }, "drone_dimensions": [ @@ -152,7 +156,9 @@ }, "properties": { "$flutter/osx_sdk": { - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } } }, @@ -172,7 +178,9 @@ "ios-16-4_14e300c", "ios-16-2_14c18" ], - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } }, "drone_dimensions": [ @@ -242,7 +250,9 @@ "ios-16-4_14e300c", "ios-16-2_14c18" ], - "sdk_version": "14e300c" + "sdk_version": "14e300c", + "toolchain_ver_arm": "JiMOaZvCH66lnGWMJdLScE-7lkUnKaw3COdWwqXmjWEC", + "toolchain_ver_intel": "Zp2HmSx7_-pu-yHqYqzoEpo0JRwNZGGwSj5V1SnLFmoC" } }, "drone_dimensions": [