-
Notifications
You must be signed in to change notification settings - Fork 468
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
fix communication with debuggserver on ios-14 #471
fix communication with debuggserver on ios-14 #471
Conversation
II was trying to test this and it seems like the debugger never connects(it copies and starts services properly). I'm testing with:
What is your setup? |
Thanks for the patch! I was working on a similar patch and I was having the same issue with the debugserver not connecting. I left a comment on the line that resolved my issue. |
This seems to be working (with the
The same app and device work when Xcode 12 launches.
|
We should probably focus on getting wired debugging working first since it’s blocking many flows. We can always open another issue for iOS14 wireless debugging. |
Hmm, cannot repeat the WIFI problem. Works fine for me on an 5th gen iPad mini. @jmagman , can you add some debug logging, which service is ios-deploy trying to start? Maybe version detection in some cases is not working properly. |
It seems there was regression on iOS 13 and older devices, as it needed dropping SSL connection now and the |
I tested the latest fixes on an Apple-Silicon DTK(Big Sur 20A5354i) and everything is working:
This looks good, great work! |
Can confirm patch works fine for me as well on iOS 14b6 iPad Air 2 and iOS 13.6.1 iPhone XS Max. Both wired and wirelessly. |
@ivanhernandez13 @Ukalnins thanks so much for this fix! |
@Ukalnins Hi I just found out you have changed the definition of This will cause |
Hi, I'm using |
Should fix #469 .
If I did not mess anything up by merging changes this should work on older and newer ios devices.
Added also the version detection for tvOS, but I don't have a device to test this.