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

Carthage support #61

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

wickwirew
Copy link

@wickwirew wickwirew commented May 15, 2018

Initially when trying to use Carthage there would be a build error. Carthage was trying to build the MessageViewController scheme in the example workspace.

This PR removes the example workspace, so to run the example project now you will have to run pod install prior. Carthage has no way, yet, to ignore specific schemes. All of the example CocoaPod specific files were added to the gitignore as well. Also I lowered the target to iOS 10. I can put it lower if you would like.

Also fixes a bug in the example project. inset was renamed to textViewInset

Let me know if you would like anything changed!

@rnystrom
Copy link
Member

@wickwirew I’d prefer to leave the examples alone so that someone can just download and run the sample vs having to run install scripts. Would you mind restoring those?

Sent with GitHawk

@wickwirew
Copy link
Author

wickwirew commented May 19, 2018

@rnystrom Sorry should elaborated more but having the Example workspace is what’s causing the issue. When Carthage looks for the MessageViewController scheme to build it’s taking the one in the Example.xcworkspace, not the MessageViewController.xcodeproj. Which is causing build errors due to a source file missing from it if I remember correctly. If there’s another solution I’m totally open to it though!

Sent with GitHawk

@rnystrom
Copy link
Member

I don’t know much about carthage. Why does it make this mistake?

Sent with GitHawk

@wickwirew
Copy link
Author

wickwirew commented May 20, 2018

Carthage will build all shared schemes in a repository. A MessageViewController scheme exists in both the CocoaPod generated workspace and the standard project. Workspaces always take precedence over projects so it will build the Example workspace one instead. Which is fine, but a new source file was added with out running pod install on the Example project, so it is not included in the target which is causing build errors.

But I suggest giving Carthage a try! Started using it a few months ago and have been very happy with how its working out.

Just a thought, but we can get the desired behavior of not having to run pod install for the Example project if we create the Example.xcworkspace ourselves and include the MessageViewController.xcodeproj manually instead of using CocoaPods to do it for us. Carthage will still build the project from the Example workspace but we won't run into the issue of the Scheme getting out of sync with the original project. Let me know if you like this solution and I can go ahead and add it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants