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 Install Error #637

Closed
riverbaymark opened this issue Sep 21, 2017 · 16 comments
Closed

Carthage Install Error #637

riverbaymark opened this issue Sep 21, 2017 · 16 comments
Assignees

Comments

@riverbaymark
Copy link

Trying to install via Carthage attempts to pull an incorrect (earlier) version of SocketRocket.

Mark

@levey
Copy link

levey commented Sep 23, 2017

I got the same issue.

@tcard tcard self-assigned this Oct 11, 2017
@tcard
Copy link
Contributor

tcard commented Oct 11, 2017

I'm looking into this, and even after upgrading the SocketRocket version it doesn't seem to work. It has some conflicts with CocoaPods, but I haven't been able to figure out how to fix them.

I'm afraid I'll have to deprioritize this, though, as they are other documented ways of installing Ably that work. I'll come back to this when I can.

@mattheworiordan
Copy link
Member

@tcard do we need to raise an issue upstream in SocketRocket?

@tcard
Copy link
Contributor

tcard commented Oct 12, 2017

@mattheworiordan The problem is with us, not with SocketRocket. Carthage and CocoaPods have a conflict, I guess due to some misconfiguration on our side. Xcode configuration is extremely convoluted, though, so it's not obvious (to me) what to do and I haven't found anything relevant over the Internet.

@mattheworiordan
Copy link
Member

@ricardopereira can you help with this?

@ricardopereira
Copy link
Contributor

Sure, I'll take a look.

@mattheworiordan
Copy link
Member

Thanks @ricardopereira

@ricardopereira
Copy link
Contributor

@riverbaymark @levey Could you test it using the carthage-support branch, please?

Cartfile example:

github "ably/ably-ios" "carthage-support"

@mattheworiordan
Copy link
Member

@riverbaymark @levey did this help?

@riverbaymark
Copy link
Author

riverbaymark commented Oct 29, 2017 via email

@riverbaymark
Copy link
Author

riverbaymark commented Oct 30, 2017 via email

@mattheworiordan
Copy link
Member

@riverbaymark thanks for trying it out, that's great news. @ricardopereira should we merge and do a release soon then? When you do, please be sure to follow the release process as we now ensure all releases go into our product-wide changelog

@ricardopereira
Copy link
Contributor

@riverbaymark Thanks!

Yes, I can merge it and do a release. @mattheworiordan Nice! Is that generated automatically?

@mattheworiordan
Copy link
Member

Nice! Is that generated automatically?

No, I take the markup from official releases at https://github.com/ably/ably-ios/releases and republish.

@riverbaymark
Copy link
Author

riverbaymark commented Oct 31, 2017 via email

@ricardopereira
Copy link
Contributor

Thanks for the suggestion.
Well, that's basically how Carthage works but it's always a nice thing to have. 👍

@tcard tcard closed this as completed Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

5 participants