Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[tool] Install the corresponding APK in flutter run #113622

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

Merged
merged 3 commits into from
Oct 19, 2022

Conversation

AlexV525
Copy link
Member

@AlexV525 AlexV525 commented Oct 18, 2022

Revert #113614. Fix #111585, #113620.

The PR stops copying app.apk for flutter run and makes the tool installs exactly the generated APK file, which corrects the console's output too. Also, SHA1 will be generated for all generated APKs in the batch.

Filename comparison (DEBUG Mode)

Install APK file Before After
No flavor app.apk app-debug.apk
Flavor configured app.apk app-flavor-debug.apk

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.
  • All existing and new tests are passing.

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Oct 18, 2022
@AlexV525 AlexV525 linked an issue Oct 18, 2022 that may be closed by this pull request
@flutter-dashboard flutter-dashboard bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Oct 18, 2022
@AlexV525 AlexV525 marked this pull request as ready for review October 18, 2022 17:51
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

@@ -114,8 +114,8 @@ void main() {

_findNextMatcherInList(
stdout,
(String line) => line.startsWith('Installing build/app/outputs/flutter-apk/app.apk...'),
'Installing build/app/outputs/flutter-apk/app.apk...',
(String line) => line.startsWith('Installing build/app/outputs/flutter-apk/app-release.apk...'),
Copy link
Contributor

Choose a reason for hiding this comment

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

For posterity, this is the new fix for #113620 that caused the previous PR to get reverted.

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 c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#112768 broke Mac_android run_release_test [tool] flutter run runs only app.apk
2 participants