Skip to content

[flutter_tools] fix failure to create ansi spinner if download needs to be retired#65797

Merged
jonahwilliams merged 1 commit intoflutter:masterfrom
jonahwilliams:fix_status_cancel
Sep 15, 2020
Merged

[flutter_tools] fix failure to create ansi spinner if download needs to be retired#65797
jonahwilliams merged 1 commit intoflutter:masterfrom
jonahwilliams:fix_status_cancel

Conversation

@jonahwilliams
Copy link
Contributor

Description

Because the Status spinner was only created on setup, the second stop would asset/crash.

Fixes #65449

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Sep 14, 2020
Copy link
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

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

LGTM

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

@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

@jmagman
Copy link
Member

jmagman commented Sep 15, 2020

Looks like a flake:

SEVERE: Exception while executing runnable io.grpc.internal.ServerImpl$JumpToApplicationThreadServerStreamListener$1HalfClosed@52ca2b18
java.lang.RuntimeException: Failed to get observatory port. Check logcat to see if the app started successfully.

@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

@jmagman
Copy link
Member

jmagman commented Sep 15, 2020

sudo waiting for tree to go green?

@jonahwilliams
Copy link
Contributor Author

Yeah, failure is unrelated. Will submit on green

@jonahwilliams jonahwilliams merged commit 6acea15 into flutter:master Sep 15, 2020
@jonahwilliams jonahwilliams deleted the fix_status_cancel branch September 15, 2020 00:55
christopherfujino pushed a commit to chris-forks/flutter that referenced this pull request Sep 25, 2020
…to be retried (flutter#65797)

Because the Status spinner was only created on setup, the second stop would asset/crash.

Fixes flutter#65449
christopherfujino added a commit that referenced this pull request Sep 25, 2020
* Deprecate VelocityTracker default constructor and added VelocityTracker.withKind constructor (#66043)

* [flutter_tools] fix failure to create ansi spinner if download needs to be retried (#65797)

* Listen to Debug VM stream to get Stdout logs from VMService (#66310)

* Update cherry picked engine

Co-authored-by: Jonah Williams <[email protected]>
Co-authored-by: Jenn Magder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[tool_crash] NoSuchMethodError: NoSuchMethodError: The method 'cancel' was called on null.Receiver: nullTried calling: cancel()

5 participants