-
Notifications
You must be signed in to change notification settings - Fork 94
Runtime crash on iOS 8 #12
Comments
@Kemcake can you confirm this? TBH I didn't test iOS versions < 9 |
+1. I'm actually even can't compile with XCode 6. Yeah, this is my fault as a developer to forget about this when deploying to AppStore.
|
@ptmt @jjshammas since I assume you both have the iOS 8.x simulator installed, can you try changing the line in your - (void)application:(UIApplication *)application performActionForShortcutItem:(NSObject *)shortcutItem completionHandler:(void (^)(BOOL succeeded)) completionHandler {
[RNQuickActionManager onQuickActionPress:shortcutItem completionHandler:completionHandler];
} I'm curious to see if that works. I'm downloading the simulator now but it's a big file so it may take a while. |
@jjshammas @ptmt at this point I'm pretty close to saying, "It just ain't gonna work", but I'm willing to be proven wrong. I spent a little while playing around with the 8.x iOS simulator and I can't get it working without removing nearly all references to @Kemcake let us know if there is something magical you did you get it working that we are missing. |
Can't we use compile-time macros? http://stackoverflow.com/a/19790136 More details on various techniques here https://developer.apple.com/library/ios/documentation/DeveloperTools/Conceptual/cross_development/Introduction/Introduction.html |
It would be nice solution if Apple can recompile apps against each SDK. As far as I understand, they don't do it, so we need a dynamic runtime checking. I'm thinking about something like this:
I hope to find the better and shorter solution, if not, will try to use this approach and test it today. Too bad that there is at 17% share by iOS8, and it's even more on our target audience. |
So, I wasted a few hours on #13, but it scares me. I would go with Weak Linking (with simple Class checking as you already have in master) or even just would set target = iOS9 (sorry iOS 8 users). |
Since we don't have any sane solutions for this issue I am going make the call that for now this project does not support older versions of iOS. If anyone wants to work on this more I'd be happy to review their work and merge it in if it doesn't add a ton of complexity. 🎱 |
If I try to run an app on iOS 8 that uses this plugin, I crash immediately with this error:
The text was updated successfully, but these errors were encountered: