-
Notifications
You must be signed in to change notification settings - Fork 158
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
Add tvOS support #187
base: master
Are you sure you want to change the base?
Add tvOS support #187
Conversation
Not sure what exactly goes wrong, but it says that
I'm also not sure why to use Rakefile while we can go with .travis.yml similar to what Alamofire has. |
Last time this was attempted it got stopped at some weird code signing issues - #175 I think switching to a similar |
@orta Sorry, it took me longer than I would expect :) The only thing I'm not quite sure about is whether I need to add libExpecta-tvOS and do we need to run tests for iOS 8 and 9? I've skipped tests for iOS 8 / 9 due to the Travis CI issue travis-ci/travis-ci#7638. Also do I have to remove rakefile and objc-build-scripts? |
Never worry about your time WRT shipping OSS. Doesn't look like anyone is being blocked by your work. I think as long as there is the ability to generate a library that can be consumed by others (@tonyarnold has been shipping those with releases) then I think that's really the only responsibility left in the Rakefile or the build scripts. |
I ran (according to the Carthage guide)
and got the following output (similar to what Tony Arnold did) |
Adding tvOS support with Xcode 8.3