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

Using RK in Extensions #124

Closed
akbsteam opened this issue Oct 10, 2016 · 8 comments
Closed

Using RK in Extensions #124

akbsteam opened this issue Oct 10, 2016 · 8 comments

Comments

@akbsteam
Copy link

akbsteam commented Oct 10, 2016

I'd like to use RK in the iMessage app (extension) I'm building but I get the following:-

linking against a dylib which is not safe for use in application extensions: ....

for both RK and Bond.

Any ideas?

@srdanrasic
Copy link
Contributor

Let me research that, I didn't try it before.

@akbsteam
Copy link
Author

Looks like from quick tests its just a case of

set the target’s “Require Only App-Extension-Safe API” build setting to Yes

https://developer.apple.com/library/content/documentation/General/Conceptual/ExtensibilityPG/ExtensionScenarios.html

Assuming the frameworks don't use anything not allowed in extensions?

@srdanrasic
Copy link
Contributor

I think ReactiveKit is good to go. I can't think of any instance where Bond uses extension unsafe APIs, but I need to double check.

@akbsteam
Copy link
Author

akbsteam commented Nov 9, 2016

It seems its ok once the target is set (Both RK and Bond)

@srdanrasic
Copy link
Contributor

Great to hear that! :)

@iandundas
Copy link
Contributor

Did this get merged though? Still seeing the issue from Bond & ReactiveKit

@srdanrasic
Copy link
Contributor

srdanrasic commented Feb 7, 2017 via email

@iandundas
Copy link
Contributor

I could definitely be mistaken, but AFAIK it asserts that this dependancy doesn't use extension-unsafe API calls. By default this is set to YES on a new app extension target, and linking this with ReactiveKit/Bond triggers a warning - I think ReactiveKit/Bond need to set that flag too.

For reference I searched around and a bunch of other libs have merged the same issue:

bignerdranch/CoreDataStack#115
SnapKit/Masonry#328
jmfieldman/Mortar#11
Haneke/HanekeSwift#253

screen shot 2017-02-08 at 09 01 57

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

No branches or pull requests

3 participants