Skip to content

Revert "[flutter_tools] listen to the nice ios-deploy tool" #68118

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 1 commit into from
Oct 14, 2020

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Oct 14, 2020

Reverts #68078

  1. We shouldn't uninstall iOS apps out from the user. If it's the last app on the phone using the development cert, users have to go back in and approve on the next install.

  2. That message is from Detach debugger when VM connection fails on iOS #68046 not the ios-deploy. That message should probably be "try quitting the app and try again" not uninstall. I'll open a PR for that.

  3. There's something Bad happening in the devicelab that isn't caused by this, it's just manifesting this way. If we uninstall the app and retry we will be hiding flakes because it really took 2 tries but we would count it as working on the first go.

…S, try uninstalling it and trying again. (#68078)"

This reverts commit 61ba7dd.
@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Oct 14, 2020
@jmagman jmagman requested a review from jonahwilliams October 14, 2020 17:04
@google-cla google-cla bot added the cla: yes label Oct 14, 2020
@jonahwilliams jonahwilliams merged commit 431c599 into master Oct 14, 2020
@jonahwilliams jonahwilliams deleted the revert-68078-try_handle_install_failure_ios branch October 14, 2020 17:07
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.

2 participants