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

Fix broken header imports #233

Closed
wants to merge 2 commits into from
Closed

Fix broken header imports #233

wants to merge 2 commits into from

Conversation

JanNash
Copy link
Contributor

@JanNash JanNash commented Sep 16, 2016

These imports prevented my Swift Framework project from building.
Unfortunately I can't give out the project for reproduction, since it's private and commercial.
If you need a repro-project, maybe I can quickly set one up. :)

@hamchapman
Copy link
Contributor

A project that reproduces this would be great! I can't seem to myself

@hamchapman
Copy link
Contributor

A customer reported an issue that sounded the same for which these changes fixed their problem. However, their problem also seems to be fixed by using the latest version of cocoapods (1.1.0 at time of writing), which you can install by running gem install cocoapods --pre. Then delete the derived data directory for your app / framework, delete the Pods directory, clean the build (CMD + SHIFT + K) then run pod install and see if the project builds.

Let me know if that helps you out, otherwise a project reproducing the error would be really useful!

@JanNash
Copy link
Contributor Author

JanNash commented Sep 22, 2016

Word. I don't encounter the problem anymore as of CP 1.0.1.
I was stuck on CP 0.39 because of RestKit and its problems with use_frameworks!
So I'd be fine if we ditched this PR. Thanks for chiming in :)

@JanNash JanNash closed this Sep 22, 2016
@JanNash JanNash deleted the fix-broken-header-imports branch September 22, 2016 15:22
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