Skip to content
This repository has been archived by the owner on Aug 22, 2019. It is now read-only.

there is no tvOS scheme #67

Closed
spunkedy opened this issue Aug 12, 2016 · 18 comments
Closed

there is no tvOS scheme #67

spunkedy opened this issue Aug 12, 2016 · 18 comments

Comments

@spunkedy
Copy link
Contributor

I am not as skilled as I wish I was with swift and xcode. However to get it working I had to rip out anything that had to do with a view controller to get it running.

I was able to use it with my tvOS app to get subscriptions and make ddp calls though. I would create a PR but I do not know the proper way to exclude files / functionality from a scheme.

@siegesmund
Copy link
Owner

There's only one thing that depends on a view controller, and that supported the zero-config OAuth. I'm in the process of removing and replacing it with instructions on how to accomplish the same thing with the Facebook iOS SDK. So, the thing that's been holding you up is on its way out, obviating the need to exclude the files.

So, I'm happy to look at a PR for tvOS. tvOS support would be great and integrating it should be easy.

@spunkedy
Copy link
Contributor Author

Once that part is out I will go through and try to get a PR going via adding the scheme. Right now I have the locally modified code working through Carthage.

@siegesmund
Copy link
Owner

I believe all of the iOS specific stuff should be out of the master branch now.

@spunkedy
Copy link
Contributor Author

I will try to check today

On Aug 14, 2016 11:54 AM, "Peter Siegesmund" notifications@github.com
wrote:

I believe all of the iOS specific stuff should be out of the master branch
now.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#67 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqKL40Dw-LJQtYJ4jeYj3RLa0BKN79zks5qf0hWgaJpZM4Jiy-4
.

@spunkedy
Copy link
Contributor Author

I tried adding in the scheme.

=== CLEAN TARGET Nimble-tvOS OF PROJECT Nimble WITH CONFIGURATION Release ===

Check dependencies
target 'Nimble-tvOS' has bitcode disabled (ENABLE_BITCODE = NO), but it is required for the 'appletvos' platform

=== BUILD TARGET Nimble-tvOS OF PROJECT Nimble WITH CONFIGURATION Release ===

Check dependencies
target 'Nimble-tvOS' has bitcode disabled (ENABLE_BITCODE = NO), but it is required for the 'appletvos' platform

With the default Cartfile I see that.

I don't know how intrinsic Nimble is to the usage. However I was able to get it my local version working with just:

  • CryptoSwift
  • SwiftWebSocket
  • XCGLogger

@siegesmund
Copy link
Owner

Nimble is for the DDP tests. I'd imagine that this may just be a matter of updating the Nimble version to the latest version.

@spunkedy
Copy link
Contributor Author

Quick/Nimble#213

@spunkedy
Copy link
Contributor Author

My latest version of carthage with homebrew is 0.16.2 trying to update

@spunkedy
Copy link
Contributor Author

I have it forked, however I cannot get Carthage to build it as a reference. Xcode builds it fine. Would you like me to push the PR over, or wait until I get carthage working. It might be a while because I am about to go on vacation.

@siegesmund
Copy link
Owner

Are you getting the same errors as before?

@spunkedy
Copy link
Contributor Author

No, I just suck right now at adjusting schemes for tvOS. If I change the whole project to use the tvOS sdk it works fine.

Learning how to make the targets appropriately for carthage to use instead of a differently named scheme is where I am at.

I will try to look at it this week a little more.

@siegesmund
Copy link
Owner

I've never made a tvOS app, and I don't use Carthage much either, though for other reasons, so I'm not sure I'm much help here.

@spunkedy
Copy link
Contributor Author

haha no worries. I will go through and look at how SwiftWebSocket does it and just mimic it for this code base. It might be a few days though just a heads up

@siegesmund
Copy link
Owner

Sounds like a good approach. Thanks for doing this!

@spunkedy
Copy link
Contributor Author

np.

siegesmund added a commit that referenced this issue Aug 18, 2016
Support tvos on CocoaPods #67
@siegesmund
Copy link
Owner

Closing this issue as I've merged the pull requests.

@spunkedy
Copy link
Contributor Author

Sounds good. When you add in the oAuth screens back in, might have to disable them from the schema/ allow for a separation

@siegesmund
Copy link
Owner

If OAuth gets back into the library, it shouldn't conflict in any way with tvOS. To the best of my knowledge, the only iOS specific portion of it was the UIViewController and the WKWebView. Neither of those will be used in a future OAuth implementation.

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

No branches or pull requests

2 participants