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

Updated podspec to includes sub spec for just ObjC #7

Merged
merged 2 commits into from Feb 25, 2016

Conversation

bencallis
Copy link
Contributor

The existing pod spec brings in the objective-c files and swift. If using this pods in a project which includes static libs then the pod can not be used as it contains swift and there has to be a framework.

This PR includes a 2 subspaces one for ObjC and one for Swift.

Example usage just for ObjC

      pod 'PMKVObserver/ObjC'

@bencallis
Copy link
Contributor Author

Solves this issue #6

end

s.subspec 'Swift' do |ss|
ss.ios.deployment_target = '8.0'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why does this re-declare the deployment target when the parent spec declares it already?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good spot shouldn't be needed.

@bencallis
Copy link
Contributor Author

@kballard That should be it sorted now 👍

lilyball added a commit that referenced this pull request Feb 25, 2016
Updated podspec to includes sub spec for just ObjC
@lilyball lilyball merged commit 2254bc3 into postmates:master Feb 25, 2016
@lilyball
Copy link
Collaborator

Thanks!

@bencallis
Copy link
Contributor Author

@kballard no problem! Can you push the new pod spec to the central pods repo?

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

2 participants