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

Add ability to pass in :products option to vendored Xcode projects #119

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chendo
Copy link
Contributor

@chendo chendo commented Aug 13, 2013

This PR adds the ability to specify a :products option to the Xcode vendor project type to match the :products option on static vendor projects.

Use case:
Cocoapods recently added dedicated targets for each pod (CocoaPods/CocoaPods#983) , then link it all together in libPods.a. However, RubyMotion attempts to -force_load in all the libraries found in the Pods build directory which includes all the libPods-RegexKitLite.a etc, so it ends up failing to link due to duplicate symbols.

I use the following with the patch below to fix the issue:

  app.vendor_project('Pods', :xcode,
                      :target        => 'Pods',
                      :products => ['libPods.a'])

@chendo
Copy link
Contributor Author

chendo commented Apr 5, 2014

This is still a problem for integrating with OS X projects using Cocoapods.

@alloy
Copy link
Member

alloy commented Jun 25, 2014

This looks good to me, I'll just have to test it manually first. Can you elaborate on how/why you need this on OS X specifically?

@chendo
Copy link
Contributor Author

chendo commented Jun 25, 2014

I’m using RubyMotion to test an Obj-C app that uses Cocoapods and I’m using the products to specify only to use the libPods.a file, otherwise it will try to include all the .a files compiled by CocoaPods and cause duplicate symbol issues. If there’s another way to solve this issue, I’d be happy to use that instead.

On Wednesday, 25 June 2014 at 5:17 am, Eloy Durán wrote:

This looks good to me, I'll just have to test it manually first. Can you elaborate on how/why you need this on OS X specifically?


Reply to this email directly or view it on GitHub (#119 (comment)).

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.

2 participants