Skip to content

Commit d882b2d

Browse files
Revert "Use CIPD to download prebuilt Dart SDKs (flutter#33345)" (flutter#33558)
This reverts commit 051eee8.
1 parent 4bff087 commit d882b2d

File tree

4 files changed

+312
-61
lines changed

4 files changed

+312
-61
lines changed

DEPS

Lines changed: 10 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -351,58 +351,6 @@ deps = {
351351

352352
# WARNING: end of dart dependencies list that is cleaned up automatically - see create_updated_flutter_deps.py.
353353

354-
# Prebuilt Dart SDK of the same revision as the Dart SDK source checkout
355-
'src/flutter/prebuilts/linux-x64/dart-sdk': {
356-
'packages': [
357-
{
358-
'package': 'flutter/dart-sdk/linux-amd64',
359-
'version': 'git_revision:'+Var('dart_revision')
360-
}
361-
],
362-
'dep_type': 'cipd',
363-
'condition': 'host_os == "linux"'
364-
},
365-
'src/flutter/prebuilts/linux-arm64/dart-sdk': {
366-
'packages': [
367-
{
368-
'package': 'flutter/dart-sdk/linux-arm64',
369-
'version': 'git_revision:'+Var('dart_revision')
370-
}
371-
],
372-
'dep_type': 'cipd',
373-
'condition': 'host_os == "linux"'
374-
},
375-
'src/flutter/prebuilts/macos-x64/dart-sdk': {
376-
'packages': [
377-
{
378-
'package': 'flutter/dart-sdk/mac-amd64',
379-
'version': 'git_revision:'+Var('dart_revision')
380-
}
381-
],
382-
'dep_type': 'cipd',
383-
'condition': 'host_os == "mac"'
384-
},
385-
'src/flutter/prebuilts/macos-arm64/dart-sdk': {
386-
'packages': [
387-
{
388-
'package': 'flutter/dart-sdk/mac-arm64',
389-
'version': 'git_revision:'+Var('dart_revision')
390-
}
391-
],
392-
'dep_type': 'cipd',
393-
'condition': 'host_os == "mac"'
394-
},
395-
'src/flutter/prebuilts/windows-x64/dart-sdk': {
396-
'packages': [
397-
{
398-
'package': 'flutter/dart-sdk/windows-amd64',
399-
'version': 'git_revision:'+Var('dart_revision')
400-
}
401-
],
402-
'dep_type': 'cipd',
403-
'condition': 'host_os == "win"'
404-
},
405-
406354
'src/third_party/colorama/src':
407355
Var('chromium_git') + '/external/colorama.git' + '@' + '799604a1041e9b3bc5d2789ecbd7e8db2e18e6b8',
408356

@@ -732,6 +680,16 @@ hooks = [
732680
'src/flutter/tools/pub_get_offline.py',
733681
]
734682
},
683+
{
684+
'name': 'Download prebuilt Dart SDK',
685+
'pattern': '.',
686+
'condition': 'download_dart_sdk',
687+
'action': [
688+
'python3',
689+
'src/flutter/tools/download_dart_sdk.py',
690+
'--fail-loudly',
691+
]
692+
},
735693
{
736694
'name': 'Download Fuchsia SDK',
737695
'pattern': '.',

build/archives/BUILD.gn

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -104,15 +104,10 @@ zip_bundle("flutter_patched_sdk") {
104104
# Flutter consumes the dart sdk as a prebuilt. Rather than regenerating
105105
# the zip file we are just copying the original file to the artifacts location.
106106
if (build_engine_artifacts && flutter_prebuilt_dart_sdk) {
107-
zip_bundle("dart_sdk_archive") {
108-
deps = []
109-
output = "dart-sdk-$full_target_platform_name.zip"
110-
files = [
111-
{
112-
source = prebuilt_dart_sdk
113-
destination = "dart-sdk"
114-
},
115-
]
107+
copy("dart_sdk_archive") {
108+
sources = [ prebuilt_dart_sdk_archive ]
109+
outputs =
110+
[ "$root_out_dir/zip_archives/dart-sdk-$full_target_platform_name.zip" ]
116111
}
117112
}
118113

common/config.gni

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,9 @@ if (flutter_prebuilt_dart_sdk) {
112112

113113
_target_prebuilt_dart_sdk_config = "$_target_os_name-$target_cpu"
114114
_host_prebuilt_dart_sdk_config = "$_host_os_name-$host_cpu"
115+
_target_prebuilt_dart_sdk_archive = "//flutter/prebuilts/dartsdk-$_target_prebuilt_dart_sdk_config-release.zip"
116+
_host_prebuilt_dart_sdk_archive =
117+
"//flutter/prebuilts/dartsdk-$_host_prebuilt_dart_sdk_config-release.zip"
115118

116119
target_prebuilt_dart_sdk =
117120
"//flutter/prebuilts/$_target_prebuilt_dart_sdk_config/dart-sdk"
@@ -124,9 +127,11 @@ if (flutter_prebuilt_dart_sdk) {
124127
if (current_toolchain == host_toolchain || is_fuchsia) {
125128
prebuilt_dart_sdk = host_prebuilt_dart_sdk
126129
prebuilt_dart_sdk_config = _host_prebuilt_dart_sdk_config
130+
prebuilt_dart_sdk_archive = _host_prebuilt_dart_sdk_archive
127131
} else {
128132
prebuilt_dart_sdk = target_prebuilt_dart_sdk
129133
prebuilt_dart_sdk_config = _target_prebuilt_dart_sdk_config
134+
prebuilt_dart_sdk_archive = _target_prebuilt_dart_sdk_archive
130135
}
131136
}
132137

0 commit comments

Comments
 (0)