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

[TIMOB-24069] Fixed pairing of ios and watch sims that has been aroun… #8667

Merged
merged 2 commits into from Dec 12, 2016
Merged

[TIMOB-24069] Fixed pairing of ios and watch sims that has been aroun… #8667

merged 2 commits into from Dec 12, 2016

Conversation

cb1kenobi
Copy link
Contributor

https://jira.appcelerator.org/browse/TIMOB-24069

[TIMOB-24069] Fixed pairing of ios and watch sims that has been around since Xcode 8 where you can't unpair certain ios and watch sims.

https://jira.appcelerator.org/browse/TIMOB-24136

[TIMOB-24136] Removed dependency on 'simctl getenv' which is broken in Xcode 8.2 beta and prevented knowning when an Xcode 8.2 sim has launched.

Updated to ioslib 1.1.0.

Major surgury to the iOS build to move the tiapp.xml validation before the --device-id validation. Also cleaned up min ios version and device family handling.

…d since Xcode 8 where you can't unpair certain ios and watch sims.

[TIMOB-24136] Removed dependency on 'simctl getenv' which is broken in Xcode 8.2 beta and prevented knowning when an Xcode 8.2 sim has launched.

Updated to ioslib 1.1.0.

Major surgury to the iOS build to move the tiapp.xml validation before the --device-id validation. Also cleaned up min ios version and device family handling.
@@ -20,6 +22,7 @@ const mkdirp = require('mkdirp');
const net = require('net');
const path = require('path');
const readPlist = require('./utilities').readPlist;
const simctl = require('./simctl');
Copy link
Collaborator

Choose a reason for hiding this comment

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

File not commited? Could not find it in node_modules/ioslib/lib/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was not committed because last October a line was added to the .gitignore to ignore everything [new] in the node_modules directory. I fixed it.

@hansemannn
Copy link
Collaborator

@cb1kenobi @cheekiatng FT looks good to me 🚀

@cheekiatng
Copy link
Contributor

APPROVED

@cheekiatng cheekiatng merged commit 9d8982d into tidev:master Dec 12, 2016
@cb1kenobi cb1kenobi deleted the timob-24069 branch December 12, 2016 07:55
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

3 participants