Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flutter: Bootstrap Flutter/Pub/Dart #3840

Merged
merged 1 commit into from
Apr 3, 2021
Merged

Conversation

sschuberth
Copy link
Member

Due to the big size (around 1.7Gb) and rare use it is more optimal to
bootstrap Flutter on demand instead of installing it in the Docker
container.

This commit implements the bootstrapping logic for Flutter (which
includes Pub) and removes Flutter from the Docker image and the CI
configurations.

With this change it is now possible to choose the Flutter version by
setting the FLUTTER_VERSION environment variable.

Co-authored-by: Martin Nonnenmacher martin.nonnenmacher@here.com
Signed-off-by: zhernovs ext-andriy.zhernovskyi@here.com

Due to the big size (around 1.7Gb) and rare use it is more optimal to
bootstrap Flutter on demand instead of installing it in the Docker
container.

This commit implements the bootstrapping logic for Flutter (which
includes Pub) and removes Flutter from the Docker image and the CI
configurations.

With this change it is now possible to choose the Flutter version by
setting the `FLUTTER_VERSION` environment variable.

Co-authored-by: Martin Nonnenmacher <martin.nonnenmacher@here.com>
Signed-off-by: zhernovs <ext-andriy.zhernovskyi@here.com>
@sschuberth sschuberth requested a review from a team as a code owner April 2, 2021 09:28
private val flutterCommand = if (Os.isWindows) "flutter.bat" else "flutter"
private val pubCommand = if (Os.isWindows) "pub.bat" else "pub"

private val flutterVersion = Os.env["FLUTTER_VERSION"] ?: "v1.12.13+hotfix.9-stable"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see this still uses Flutter v1.12.13 which current stable is 2.0.3 see https://flutter.dev/docs/development/tools/sdk/releases so we should upgrade else scans will likely fail as the Flutter community is using more and more 2.x.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the intention of this PR was not to upgrade Flutter, but to implement the bootstrap mechanism without any further changes to keep things as simple as possible. We can do the upgrade as a follow-up.

@sschuberth
Copy link
Member Author

Merging this without waiting for another review as this is just a copy of #3543 which has already been reviewed.

@sschuberth sschuberth merged commit edbb46c into master Apr 3, 2021
@sschuberth sschuberth deleted the bootstrap-flutter-2 branch April 3, 2021 07:40
@mnonnenmacher
Copy link
Member

Thanks for handling this @sschuberth.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants