Skip to content

Added iOS 8 compatibility#4

Open
mootjeuh wants to merge 2 commits into
a1anyip:masterfrom
mootjeuh:ios8
Open

Added iOS 8 compatibility#4
mootjeuh wants to merge 2 commits into
a1anyip:masterfrom
mootjeuh:ios8

Conversation

@mootjeuh

Copy link
Copy Markdown

Apparently XPCConnectionHasEntitlement is not the way to go anymore so it's now hooking BSAuditTokenTaskHasEntitlement on iOS 8, as per the wiki.

@MP0w

MP0w commented Nov 14, 2014

Copy link
Copy Markdown

Why you use [[NSProcessInfo processInfo] respondsToSelector:@selector(operatingSystemVersion)] ?
Wouldn't be better to check the respondToSelector of the selector you are going to call?

isassertiond should be camel case like the rest of the names.

Too bad it now require a reboot after install, user are going to hate this.

@mootjeuh

Copy link
Copy Markdown
Author

It was a quick copy and paste, but it boils down to the same thing operatingSystemVersion is only available in iOS 8 and applicationWithDisplayIdentifier only in iOS 7, guess I'll change it when I update for the surrounding naming conventions.

And yeah it is kinda irritating to have to reboot, but it's the way of the road I guess.

@MP0w

MP0w commented Nov 14, 2014

Copy link
Copy Markdown

"...operatingSystemVersion is only available in iOS 8 ."
Yes, but it doesn't make sense to assume this while you can just check for the right selector =)

@mootjeuh

Copy link
Copy Markdown
Author

Fine there you go you drama queen :)

@myeyesareblind

Copy link
Copy Markdown

@a1anyip : will you merge this?

@ianb821

ianb821 commented Apr 12, 2015

Copy link
Copy Markdown

@mootjeuh Great work on the update. Looking forward to having the update widely available on Cydia.

@xuanxt

xuanxt commented Oct 14, 2015

Copy link
Copy Markdown

It can support IOS9?

@NSExceptional

Copy link
Copy Markdown

Hello?

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.

6 participants