Build and release our own gen_snapshots in the engine CI

[bbrto21]

Originally posted by @swift-kim in #219 (comment)

Target host

• Windows
• MacOS X

[swift-kim]

https://github.com/flutter/flutter/wiki/Setting-up-the-Engine-development-environment

The problematic part seems to be this part:

• On Windows:
  • Visual Studio 2017 or later (required for non-Googlers only).
  • Windows 10 SDK (required for non-Googlers only).
    • Be sure to install the "Debugging Tools for Windows" feature.

Can these prerequisites be installed from the command line on GitHub-hosted instances?

[bbrto21]

• Visual Studio 2017 or later (required for non-Googlers only).

Visual Studio 2022 was installed on my Windows machine, but the tool said it is not a supported version :(

[bwikbs]

Can these prerequisites be installed from the command line on GitHub-hosted instances?

It seems to be difficult to get instance what is satisfied that requirement.... May be self hosted runner?

[swift-kim]

Build commands for Windows:

# Targeting tizen-arm-release.
python .\flutter\tools\gn --target-os=linux --linux-cpu=arm --no-goma --runtime-mode=release
ninja -C .\out\linux_release_arm gen_snapshot

[WonyoungChoi]

Github hosted runner provides VS environment 🎉 https://github.com/actions/virtual-environments/blob/main/images/win/Windows2019-Readme.md#visual-studio-enterprise-2019

[bwikbs]

Github hosted runner provides VS environment 🎉 https://github.com/actions/virtual-environments/blob/main/images/win/Windows2019-Readme.md#visual-studio-enterprise-2019

It's amazing!! github is really generous!

[swift-kim]

The tool side work is done by flutter-tizen/flutter-tizen#296. Now the only work to do is to add gen_snapshot executable for each platform in each of our zipped artifacts (tizen-arm-release.zip and etc.).

e.g. tizen-arm-release.zip should contain:

• libflutter_*.so
• linux-x64/gen_snapshot
• windows-x64/gen_snapshot.exe
• darwin-x64/gen_snapshot

The better design should be downloading only files that are relevant for the host platform (e.g. only linux-x64 but no darwin-x64 on Linux). How can we accomplish this?

[swift-kim]

@WonyoungChoi

the artifacts will be separated to download only the artifact required for the host platform

How this will be done is exactly what I was asking in the above comment. Ideally, gen_snapshot should be placed in the corresponding engine artifact directory as follows:

$ tree flutter/bin/cache/artifacts/engine/tizen-*-release
flutter/bin/cache/artifacts/engine/tizen-arm-release
├── libflutter_engine.so
├── libflutter_tizen_common.so
├── libflutter_tizen_mobile.so
├── libflutter_tizen_tv.so
├── libflutter_tizen_wearable.so
└── linux-x64
    └── gen_snapshot
flutter/bin/cache/artifacts/engine/tizen-arm64-release
├── libflutter_engine.so
├── libflutter_tizen_common.so
├── libflutter_tizen_mobile.so
├── libflutter_tizen_tv.so
├── libflutter_tizen_wearable.so
└── linux-x64
    └── gen_snapshot

Are you thinking about zipping linux-x64/gen_snapshot separately from tizen-arm-release.zip? What name will the zip file be?

[WonyoungChoi]

In the case of Android, the artifact name (or path) is android-arm-release/windows-x64.zip. maybe we should use the name like tizen-arm-release-windows-x64.zip because Github CI does not support the directory path for the artifact.

[swift-kim]

Now the default engine branch is flutter-2.8.1-tizen. I'll discard the 2.8.0 pre-release when the 2.8.1 artifacts are ready.

[swift-kim]

tizen-arm-release-windows-x64.zip

@WonyoungChoi What files will be included in this archive?

[WonyoungChoi]

tizen-arm-release-windows-x64.zip

@WonyoungChoi What files will be included in this archive?

I think only the gen_snapshot binary.

tizen-arm-release-windows-x64.zip
==>
tizen-arm-release/
└── windows-x64
    └── gen_snapshot