Skip to content

Conversation

@gmackall
Copy link
Member

@gmackall gmackall commented Sep 10, 2024

Sum of changes is:
cherry pick of #154757
Merge
manual application of of #154945
manual application of #154967

Copied form below from automatic label (this is a cherry pick of #154757).

Issue Link:

What is the link to the issue this cherry-pick is addressing?

Fixes #154371.

Changelog Description:

Explain this cherry pick in one line that is accessible to most Flutter developers. See best practices for examples

Fixes a crash when building Flutter modules for android with flutter build aar, when using Android Gradle Plugin 8.0+.

Impact Description:

What is the impact (ex. visual jank on Samsung phones, app crash, cannot ship an iOS app)? Does it impact development (ex. flutter doctor crashes when Android Studio is installed), or the shipping production app (the app crashes on launch)

Fixes the mentioned crash when building a module as an aar with Android Gradle Plugin 8.0+.

Workaround:

Is there a workaround for this issue?

No.

Risk:

What is the risk level of this cherry-pick?

  • Low
  • Medium
  • High

Test Coverage:

Are you confident that your fix is well-tested by automated tests?

  • Yes
  • No

Validation Steps:

What are the steps to validate that this fix works?

Build a module:

  1. flutter create foobar --template=module
  2. cd foobar
  3. Edit the .android/build.gradle to use AGP 8.1.
  4. flutter run/flutter build aar

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@flutter-dashboard
Copy link

This pull request was opened from and to a release candidate branch. This should only be done as part of the official Flutter release process. If you are attempting to make a regular contribution to the Flutter project, please close this PR and follow the instructions at Tree Hygiene for detailed instructions on contributing to Flutter.

Reviewers: Use caution before merging pull requests to release branches. Ensure the proper procedure has been followed.

@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Sep 10, 2024
@gmackall gmackall changed the title [CP-stable] Cherry pick "Upgrade module template AGP version" [CP-stable] Cherry pick "Fix flutter build aar for modules that use a plugin" Sep 10, 2024
@gmackall gmackall marked this pull request as ready for review September 10, 2024 20:24
@gmackall

This comment was marked as outdated.

@gmackall gmackall requested a review from reidbaker September 10, 2024 20:41
@reidbaker
Copy link
Contributor

Sum of changes is: cherry pick of #154757 Merge manual application of of #154945 manual application of #154967

This comment should be at the top of the pull request description.

Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

@itsjustkevin itsjustkevin added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 11, 2024
@auto-submit auto-submit bot merged commit 6553713 into flutter:flutter-3.24-candidate.0 Sep 11, 2024
auto-submit bot pushed a commit that referenced this pull request Sep 11, 2024
# Flutter stable 3.24.3 Framework

## Scheduled Cherrypicks
- #154380
- #154720
- #154944
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 13, 2024
Gustl22 pushed a commit to Gustl22/flutter that referenced this pull request Nov 13, 2024
@amaralphp amaralphp mentioned this pull request Mar 27, 2025
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants