-
Notifications
You must be signed in to change notification settings - Fork 30
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 doesn't actually work #94
Comments
Thanks for reporting this, @drewcrawford! I'll look into it right away. |
Ok @drewcrawford, I pushed a fix in be92e6e, can you try again but this time by pointing Carthage at the So your Cartfile should look like this:
|
Better, but still fails:
|
Sorry, forgot to ask you to run |
It's kind of tricky I guess, we need to run |
So, the right answer here I think depends on what you are trying to accomplish. If the goal here is to support installing via carthage in a useful way,
1 is suboptimal (but at least fairly likely), 2 is unlikely and false in my case. The docs could of course tell me how to solve 2, but I question the value of "supporting" carthage by requiring the user to first configure cocoapods in a special way. I personally use carthage to avoid cocoapods, if I can't avoid cocoapods I might as well use cocoapods. If you are happy with cocoapods being a build dependency I might suggest just not trying to support carthage. People like me can either suck it up and use cocoapods or be pushed to something like Rome. |
I see what you mean, but I'd still prefer to provide at least some support for Carthage users (like having a Cartfile and thus our dependencies explicitly stated.) Having to run I wonder if someone has figured out a way to support both properly. |
I am really not sure this is worth it. I'll remove Carthage support for now so that we're not lying to window-shoppers, and if anyone can come up with a good way to support it again, I'll be more than happy to merge a PR. I think that's the most honest thing I can do right now. |
One can work around this issue with
$ cd Carthage/Checkouts/XcodeServerSDK/ $ pod install
However it then fails with a new error
To work around this, I had to remove iOS/watch targets as you can see in my fork. I don't need them, so that wasn't a big deal.
The text was updated successfully, but these errors were encountered: