Skip to content

Commit

Permalink
[Tizen] Disable download fuchsia sdk
Browse files Browse the repository at this point in the history
The size of fuchsia sdk has increased from 2.3G to 6.4G.
As a result, a no memory error occurs in CI.
So apply some of this commit.
flutter@1e199ce
Add the download_fuchsia_deps flag and set this flag to False in ci.
  • Loading branch information
JSUYA committed Mar 19, 2024
1 parent 21180fd commit 27e5158
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ jobs:
- name: Run gclient sync
run: |
gclient config --name=src/flutter --unmanaged https://github.com/flutter-tizen/engine
gclient setdep --var=download_dart_sdk=False --var=download_android_deps=False --deps-file=src/flutter/DEPS
gclient setdep --var=download_fuchsia_deps=False --var=download_dart_sdk=False --var=download_android_deps=False --deps-file=src/flutter/DEPS
gclient sync -v --no-history --shallow
- name: Build
Expand Down
11 changes: 9 additions & 2 deletions DEPS
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,13 @@ vars = {
# Checkout Linux dependencies only when building on Linux.
'download_linux_deps': 'host_os == "linux"',

# Checkout Fuchsia dependencies only on Linux. This is the umbrella flag which
# controls the behavior of all fuchsia related flags. I.e. any fuchsia related
# logic or condition may not work if this flag is False.
# TODO(zijiehe): Make this condition more strict to only download fuchsia
# dependencies when necessary: b/40935282
'download_fuchsia_deps': 'host_os == "linux"',

# Downloads the fuchsia SDK as listed in fuchsia_sdk_path var. This variable
# is currently only used for the Fuchsia LSC process and is not intended for
# local development.
Expand Down Expand Up @@ -938,7 +945,7 @@ deps = {
'version': 'I-3hiLjX2DDy6mu22Q51o4ktda9zg0wZWdk-QolLicoC'
}
],
'condition': 'host_os == "linux" and not download_fuchsia_sdk',
'condition': 'download_fuchsia_deps and not download_fuchsia_sdk',
'dep_type': 'cipd',
},

Expand Down Expand Up @@ -1032,7 +1039,7 @@ hooks = [
{
'name': 'Download Fuchsia SDK',
'pattern': '.',
'condition': 'download_fuchsia_sdk',
'condition': 'download_fuchsia_deps and download_fuchsia_sdk',
'action': [
'python3',
'src/flutter/tools/download_fuchsia_sdk.py',
Expand Down

0 comments on commit 27e5158

Please sign in to comment.