Skip to content

Conversation

@cbracken
Copy link
Member

@cbracken cbracken commented Jun 19, 2017

This reverts commit b2909a2.

This resubmits the following patches:

  1. Use Xcode instruments to list devices (Use Xcode instruments to list devices #10801)
    Eliminates the dependency on idevice_id from libimobiledevice. Instead,
    uses Xcode built-in functionality.

  2. Make device discovery asynchronous (Make device discovery asynchronous #10803)
    Migrates DeviceDiscovery.devices and all device-specific lookup to be
    asynchronous.

The above patches were reverted due to incomplete Xcode installs on the
build bots. This has now been remedies.

This reverts commit b2909a2.

This resubmits the following patches:

1. Use Xcode instruments to list devices (flutter#10801)
Eliminates the dependency on idevice_id from libimobiledevice. Instead,
uses Xcode built-in functionality.

2. Make device discovery asynchronous (flutter#10803)
Migrates DeviceDiscovery.devices and all device-specific lookup to be
asynchronous.
@cbracken cbracken requested a review from tvolkert June 19, 2017 19:49
@cbracken cbracken merged commit 1d9f009 into flutter:master Jun 19, 2017
@cbracken cbracken deleted the revert-revert branch June 19, 2017 20:15
@cbracken
Copy link
Member Author

Related #10602, #10633

gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jul 1, 2017
This reverts commit b2909a2.

This resubmits the following patches:

1. Use Xcode instruments to list devices (flutter#10801)
Eliminates the dependency on idevice_id from libimobiledevice. Instead,
uses Xcode built-in functionality.

2. Make device discovery asynchronous (flutter#10803)
Migrates DeviceDiscovery.devices and all device-specific lookup to be
asynchronous.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants