-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Support Carthage #79
Support Carthage #79
Conversation
Can this be merged? I've checked out @cybertk's branch and it builds a framework successfully. |
Where can I find Pinterest.app? I will give it a shot. |
@raphaelschaad I'm not sure what you mean by Pinterest.app – I don't work at Pinterest – just trying to help out with Carthage support on PINRemoteImage. |
Whoops, I think I caused some confusion. I'm trying to help @dcaunt add Carthage support to https://github.com/pinterest/PINRemoteImage. But that depends on FLAnimatedImage getting Carthage support currently. I'm happy to run any tests. Just to be sure, you'd like me to make sure FLAnimatedImage still builds and runs with this branch in the Pinterest app? I can definitely test that. We don't actually use Carthage (yet?), so I won't easily be able to test that this actually enables Carthage support :) |
So I tested with You could additionally verify that the framework is correct by modifying the demo project and dragging in the resulting |
@dcaunt I was able to run the demo project successfully powered by the .framework. However, the .framework was built with DEBUG 0 and hence doesn't respond to calls the demo project makes use of. Is it possible to build a debug .framework with Carthage? As a side note for myself, this helped resolve an initial runtime error I got: Alamofire/Alamofire#101 |
@raphaelschaad You mean that we need set DEBUG=1 for the distributed framework? Or just set it for demo project only? |
Just for testing the demo project. I was curious whether there is a way to build a DEBUG=1 framework through Carthage, without modifying the code/project itself. PS: It would be great if you get the chance to quickly look at #79 (comment) – thanks! |
Current |
I see there is no testing target in |
Great, --configuration Debug was what I was looking for, thanks. By testing I mean verifying that the built framework works correctly by dragging it from Carthage/Build into the demo project and run it. It worked well. Let's 🚢 this. Thanks! One remaining post-merge thing I'd like to discuss is the issue with the framework version number: #79 (comment) |
@raphaelschaad Great, thank you for taking the time to run through this and merge the PR. My understanding with the version number is that it is fine to leave the version as |
@dcaunt Sure! Thank you. I read up on how Carthage versioning works and it seems tagging master as I use to will work well. @cybertk I'll include your change in the change log for the next version.
Please let me know your full name if you'd like that to show up in the change log instead (so far it's all full names but I don't mind either way). Could you also fill out Flipboard’s Apache-style Individual Contributor License Agreement (CLA)? It's standard and quick to fill out. Thanks! https://docs.google.com/forms/d/1gh9y6_i8xFn6pA15PqFeye19VqasuI9-bGp_e0owy74/viewform |
CLA is signed and my full name is Quanlong He |
@raphaelschaad When 1.0.9 hits it would be nice to provide a prebuilt framework for Carthage. Carthage can download prebuilt frameworks rather than checking out source and building, if an archive is attached to the release (example from Mantle). You can build the archive for attaching to a release like this:
|
Before this PR is merged, add following line into your
Cartfile
to use with Carthage,