Fix build mode not propagated in android native asset build by knopp · Pull Request #144550 · flutter/flutter

knopp · 2024-03-04T17:34:59Z

This fixes bug where build mode is hardcoded to debug while building native assets for Android.

Pre-launch Checklist

I read the Contributor Guide and followed the process outlined there for submitting PRs.
I read the Tree Hygiene wiki page, which explains my responsibilities.
I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
I signed the CLA.
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is test-exempt.
I followed the breaking change policy and added Data Driven Fixes where supported.
All existing and new tests are passing.

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

knopp · 2024-03-04T17:35:46Z

@dcharkes

dcharkes

Thanks @knopp! ❤️

dcharkes · 2024-03-04T17:40:11Z

Please add a PR description as well.

knopp · 2024-03-04T17:43:09Z

Please add a PR description as well.

My bad. Fixed. Feel free to edit if this is not descriptive enough.

…lutter/flutter#144550)

flutter/flutter@65cd84b...3b5a2ec 2024-03-05 [email protected] Roll Flutter Engine from 17a4b66e0729 to 8916bb32b7b8 (1 revision) (flutter/flutter#144615) 2024-03-05 [email protected] Roll Flutter Engine from d514a302cdff to 17a4b66e0729 (8 revisions) (flutter/flutter#144607) 2024-03-05 [email protected] Do not shorten native assets framework names (flutter/flutter#144568) 2024-03-05 [email protected] Restorable CupertinoTextFormFieldRow (flutter/flutter#144541) 2024-03-05 [email protected] Updated the TextButton image button example artwork (flutter/flutter#144583) 2024-03-05 [email protected] Roll Flutter Engine from 728aa4ef05d4 to d514a302cdff (2 revisions) (flutter/flutter#144581) 2024-03-05 [email protected] Roll Flutter Engine from 62cf93be60b2 to 728aa4ef05d4 (3 revisions) (flutter/flutter#144578) 2024-03-05 [email protected] Disable super flakey impeller test. (flutter/flutter#144573) 2024-03-04 [email protected] Roll Flutter Engine from 40018811ec0e to 62cf93be60b2 (1 revision) (flutter/flutter#144572) 2024-03-04 [email protected] Roll Flutter Engine from 0d8588b1de7a to 40018811ec0e (6 revisions) (flutter/flutter#144570) 2024-03-04 [email protected] Print warning and exit when iOS device is unpaired (flutter/flutter#144551) 2024-03-04 [email protected] Roll Flutter Engine from a31209171949 to 0d8588b1de7a (1 revision) (flutter/flutter#144558) 2024-03-04 [email protected] Remove unnecessary (and the only) `RenderObject.markParentNeedsLayout` override (flutter/flutter#144466) 2024-03-04 [email protected] Fix text color for default CupertinoContextMenuAction (flutter/flutter#144542) 2024-03-04 [email protected] Add missing parameter to `TableBorder.symmetric`, and improve class constructors (flutter/flutter#144279) 2024-03-04 [email protected] Fix build mode not propagated in android native asset build (flutter/flutter#144550) 2024-03-04 [email protected] Roll Packages from a9c68b8 to 0625827 (3 revisions) (flutter/flutter#144556) 2024-03-04 [email protected] Roll Flutter Engine from afd2d9f9421d to a31209171949 (1 revision) (flutter/flutter#144554) 2024-03-04 [email protected] Doc fix for `DropdownButtonFormField.value` (flutter/flutter#144427) 2024-03-04 [email protected] Fix Small Typo in Skia_Client Doc Comment (flutter/flutter#144490) 2024-03-04 [email protected] Roll Flutter Engine from 2338a557e018 to afd2d9f9421d (1 revision) (flutter/flutter#144549) 2024-03-04 [email protected] [Windows] Update keyboard modifiers link (flutter/flutter#144426) 2024-03-04 [email protected] Roll Flutter Engine from 9cbca80e2196 to 2338a557e018 (2 revisions) (flutter/flutter#144545) 2024-03-04 [email protected] Roll Flutter Engine from c52b3033eca6 to 9cbca80e2196 (1 revision) (flutter/flutter#144523) 2024-03-04 [email protected] Roll Flutter Engine from 024ea680ceed to c52b3033eca6 (1 revision) (flutter/flutter#144522) 2024-03-03 [email protected] Roll Flutter Engine from 2706c732b22a to 024ea680ceed (1 revision) (flutter/flutter#144519) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

…lutter/flutter#144550)

flutter/flutter@65cd84b...3b5a2ec 2024-03-05 [email protected] Roll Flutter Engine from 17a4b66e0729 to 8916bb32b7b8 (1 revision) (flutter/flutter#144615) 2024-03-05 [email protected] Roll Flutter Engine from d514a302cdff to 17a4b66e0729 (8 revisions) (flutter/flutter#144607) 2024-03-05 [email protected] Do not shorten native assets framework names (flutter/flutter#144568) 2024-03-05 [email protected] Restorable CupertinoTextFormFieldRow (flutter/flutter#144541) 2024-03-05 [email protected] Updated the TextButton image button example artwork (flutter/flutter#144583) 2024-03-05 [email protected] Roll Flutter Engine from 728aa4ef05d4 to d514a302cdff (2 revisions) (flutter/flutter#144581) 2024-03-05 [email protected] Roll Flutter Engine from 62cf93be60b2 to 728aa4ef05d4 (3 revisions) (flutter/flutter#144578) 2024-03-05 [email protected] Disable super flakey impeller test. (flutter/flutter#144573) 2024-03-04 [email protected] Roll Flutter Engine from 40018811ec0e to 62cf93be60b2 (1 revision) (flutter/flutter#144572) 2024-03-04 [email protected] Roll Flutter Engine from 0d8588b1de7a to 40018811ec0e (6 revisions) (flutter/flutter#144570) 2024-03-04 [email protected] Print warning and exit when iOS device is unpaired (flutter/flutter#144551) 2024-03-04 [email protected] Roll Flutter Engine from a31209171949 to 0d8588b1de7a (1 revision) (flutter/flutter#144558) 2024-03-04 [email protected] Remove unnecessary (and the only) `RenderObject.markParentNeedsLayout` override (flutter/flutter#144466) 2024-03-04 [email protected] Fix text color for default CupertinoContextMenuAction (flutter/flutter#144542) 2024-03-04 [email protected] Add missing parameter to `TableBorder.symmetric`, and improve class constructors (flutter/flutter#144279) 2024-03-04 [email protected] Fix build mode not propagated in android native asset build (flutter/flutter#144550) 2024-03-04 [email protected] Roll Packages from a9c68b8 to 0625827 (3 revisions) (flutter/flutter#144556) 2024-03-04 [email protected] Roll Flutter Engine from afd2d9f9421d to a31209171949 (1 revision) (flutter/flutter#144554) 2024-03-04 [email protected] Doc fix for `DropdownButtonFormField.value` (flutter/flutter#144427) 2024-03-04 [email protected] Fix Small Typo in Skia_Client Doc Comment (flutter/flutter#144490) 2024-03-04 [email protected] Roll Flutter Engine from 2338a557e018 to afd2d9f9421d (1 revision) (flutter/flutter#144549) 2024-03-04 [email protected] [Windows] Update keyboard modifiers link (flutter/flutter#144426) 2024-03-04 [email protected] Roll Flutter Engine from 9cbca80e2196 to 2338a557e018 (2 revisions) (flutter/flutter#144545) 2024-03-04 [email protected] Roll Flutter Engine from c52b3033eca6 to 9cbca80e2196 (1 revision) (flutter/flutter#144523) 2024-03-04 [email protected] Roll Flutter Engine from 024ea680ceed to c52b3033eca6 (1 revision) (flutter/flutter#144522) 2024-03-03 [email protected] Roll Flutter Engine from 2706c732b22a to 024ea680ceed (1 revision) (flutter/flutter#144519) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

flutter/flutter@65cd84b...3b5a2ec 2024-03-05 [email protected] Roll Flutter Engine from 17a4b66e0729 to 8916bb32b7b8 (1 revision) (flutter/flutter#144615) 2024-03-05 [email protected] Roll Flutter Engine from d514a302cdff to 17a4b66e0729 (8 revisions) (flutter/flutter#144607) 2024-03-05 [email protected] Do not shorten native assets framework names (flutter/flutter#144568) 2024-03-05 [email protected] Restorable CupertinoTextFormFieldRow (flutter/flutter#144541) 2024-03-05 [email protected] Updated the TextButton image button example artwork (flutter/flutter#144583) 2024-03-05 [email protected] Roll Flutter Engine from 728aa4ef05d4 to d514a302cdff (2 revisions) (flutter/flutter#144581) 2024-03-05 [email protected] Roll Flutter Engine from 62cf93be60b2 to 728aa4ef05d4 (3 revisions) (flutter/flutter#144578) 2024-03-05 [email protected] Disable super flakey impeller test. (flutter/flutter#144573) 2024-03-04 [email protected] Roll Flutter Engine from 40018811ec0e to 62cf93be60b2 (1 revision) (flutter/flutter#144572) 2024-03-04 [email protected] Roll Flutter Engine from 0d8588b1de7a to 40018811ec0e (6 revisions) (flutter/flutter#144570) 2024-03-04 [email protected] Print warning and exit when iOS device is unpaired (flutter/flutter#144551) 2024-03-04 [email protected] Roll Flutter Engine from a31209171949 to 0d8588b1de7a (1 revision) (flutter/flutter#144558) 2024-03-04 [email protected] Remove unnecessary (and the only) `RenderObject.markParentNeedsLayout` override (flutter/flutter#144466) 2024-03-04 [email protected] Fix text color for default CupertinoContextMenuAction (flutter/flutter#144542) 2024-03-04 [email protected] Add missing parameter to `TableBorder.symmetric`, and improve class constructors (flutter/flutter#144279) 2024-03-04 [email protected] Fix build mode not propagated in android native asset build (flutter/flutter#144550) 2024-03-04 [email protected] Roll Packages from 0d89f53 to 65ead29 (3 revisions) (flutter/flutter#144556) 2024-03-04 [email protected] Roll Flutter Engine from afd2d9f9421d to a31209171949 (1 revision) (flutter/flutter#144554) 2024-03-04 [email protected] Doc fix for `DropdownButtonFormField.value` (flutter/flutter#144427) 2024-03-04 [email protected] Fix Small Typo in Skia_Client Doc Comment (flutter/flutter#144490) 2024-03-04 [email protected] Roll Flutter Engine from 2338a557e018 to afd2d9f9421d (1 revision) (flutter/flutter#144549) 2024-03-04 [email protected] [Windows] Update keyboard modifiers link (flutter/flutter#144426) 2024-03-04 [email protected] Roll Flutter Engine from 9cbca80e2196 to 2338a557e018 (2 revisions) (flutter/flutter#144545) 2024-03-04 [email protected] Roll Flutter Engine from c52b3033eca6 to 9cbca80e2196 (1 revision) (flutter/flutter#144523) 2024-03-04 [email protected] Roll Flutter Engine from 024ea680ceed to c52b3033eca6 (1 revision) (flutter/flutter#144522) 2024-03-03 [email protected] Roll Flutter Engine from 2706c732b22a to 024ea680ceed (1 revision) (flutter/flutter#144519) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Fix build mode not propagated in android native asset build

30e55d3

github-actions Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Mar 4, 2024

knopp requested a review from dcharkes March 4, 2024 17:35

dcharkes approved these changes Mar 4, 2024

View reviewed changes

dcharkes requested a review from stuartmorgan-g March 4, 2024 17:39

knopp merged commit 1d7f4a9 into flutter:master Mar 4, 2024

knopp deleted the fix_android_build_mode branch March 4, 2024 20:11

engine-flutter-autoroll mentioned this pull request Mar 5, 2024

Roll Flutter from 65cd84b58885 to df2b36045302 (24 revisions) flutter/packages#6266

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2024

1d7f4a9 Fix build mode not propagated in android native asset build (f…

abf0915

…lutter/flutter#144550)

engine-flutter-autoroll mentioned this pull request Mar 5, 2024

Roll Flutter from 65cd84b58885 to a47af8911920 (25 revisions) flutter/packages#6268

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2024

1d7f4a9 Fix build mode not propagated in android native asset build (f…

0e801c0

…lutter/flutter#144550)

engine-flutter-autoroll mentioned this pull request Mar 5, 2024

Roll Flutter from 65cd84b58885 to 3b5a2ecf992d (26 revisions) flutter/packages#6269

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 5, 2024

1d7f4a9 Fix build mode not propagated in android native asset build (f…

2af6058

…lutter/flutter#144550)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 14, 2024

1d7f4a9 Fix build mode not propagated in android native asset build (f…

d977abe

…lutter/flutter#144550)

Navigation Menu

Repository navigation

Fix build mode not propagated in android native asset build - #144550

Fix build mode not propagated in android native asset build#144550
knopp merged 1 commit into
flutter:masterfrom
knopp:fix_android_build_mode

knopp commented Mar 4, 2024 •

edited

Loading

Uh oh!

knopp commented Mar 4, 2024

Uh oh!

dcharkes left a comment

Uh oh!

dcharkes commented Mar 4, 2024

Uh oh!

knopp commented Mar 4, 2024 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

knopp commented Mar 4, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pre-launch Checklist

Uh oh!

knopp commented Mar 4, 2024

Uh oh!

dcharkes left a comment

Choose a reason for hiding this comment

Uh oh!

dcharkes commented Mar 4, 2024

Uh oh!

knopp commented Mar 4, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

knopp commented Mar 4, 2024 •

edited

Loading

knopp commented Mar 4, 2024 •

edited

Loading