Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

Add setting for category support #64

Closed
wants to merge 1 commit into from
Closed

Add setting for category support #64

wants to merge 1 commit into from

Conversation

xtaria
Copy link

@xtaria xtaria commented Oct 30, 2013

Under "Create the Static Library Target", I added Step 6 to include support for Objective C Categories

Under "Create the Static Library Target", I added Step 6 to include support for Objective C Categories
@wilmarvh
Copy link

This saved me, thanks.

+1 for merge

@jomisj
Copy link

jomisj commented May 11, 2014

Helped me. Thanks

@jverkoey
Copy link
Owner

Will merge if we can get some clarification on the problem this addresses.

@jomisj
Copy link

jomisj commented May 12, 2014

Here is a detailed explanation of the issue:

Lets say, I have created a category for a class (say, NSString) in my framework project and made it a public header. The user won’t be able to use the category methods unless the build setting of Framework project is changed as @xtaria has mentioned. Without these build settings, the app crashes saying unknown selector for NSString.

My case a bit more different:
In my case, I had the OAuth2Client project added as a subproject of my framework project. But the Third Party app using my library/framework was crashing when my library tried to call any category methods.

More specifically, my third party app was crashing saying unknown selector in NSURL because my library was calling a method in NSURL+NXOAuth2.

So I had to change the OAuth2Client’s (subproject of my framework) build setting (“Perform Single-Object” to YES).

@jverkoey
Copy link
Owner

@jverkoey
Copy link
Owner

I'm asking because I want to minimize the number of non-default flags encouraged by this guide. Solutions in code are preferred because they solve the same problem in more situations.

@jomisj
Copy link

jomisj commented May 12, 2014

I tested solution mentioned in NimbusKitBasics. But it did not solve the issue at hand.

@jverkoey jverkoey closed this in c78f2e2 May 13, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants