Skip to content
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

Add support for iPhone and iPad debugging over USB #1734

Merged
merged 8 commits into from
Jul 4, 2020

Conversation

citizenmatt
Copy link
Member

Rider can already debug iOS devices over wifi. This PR adds support for debugging via USB.

  • Loads Unity's native extension to enumerate devices, and lists all connected devices in the "Attach to Unity Process" dialog. The list is refreshed while the dialog is open. The native extension does not give much information - it simply provides device and product ID. This can be used to look up a device type (iPad/iPhone), but there are no other details such as model number (e.g. iPhone 7) or device name (Matt's iPhone 7). There is also no indication that there is a Unity application running on the device.
  • The native extension doesn't unload cleanly, so must be used out of process - a helper .exe has been added to do this. The helper .exe simply loads the native library and starts polling for devices. The "Attach to Unity Process" dialog starts the process when the dialog is first shown, listens to its output, and gracefully shuts it down when the dialog is closed.
  • When the debugger is launched, it will load the native extension and set up a proxy from a local socket (hard coded as 12000, as per Unity's own debugger plugins) to port 56000 on the device (again hard coded)

This PR does not address USB debugging for Android.

Screenshot 2020-07-03 at 9 33 37 pm

@citizenmatt citizenmatt added this to the Rider 2020.2 milestone Jul 4, 2020
@citizenmatt citizenmatt self-assigned this Jul 4, 2020
@citizenmatt citizenmatt merged commit 5ed619c into net202 Jul 4, 2020
@citizenmatt citizenmatt deleted the feature/ios-usb-debugging branch July 4, 2020 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant