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

Add tvos #403

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add tvos #403

wants to merge 3 commits into from

Conversation

djnews24
Copy link
Contributor

@djnews24 djnews24 commented Mar 8, 2018

Hi,

tvos is missing in podspec. it works very good on tvos btw.

I know the Reachability cocopods version missing the tvos but they added it already on their github repo 2 years ago.

A correct PodFile looks like this to get it working. Reachability need installed before FreeStreamer

platform :ios, '9.0'
inhibit_all_warnings!

def shared_pods_app
pod 'Reachability', :git => 'https://github.com/tonymillion/Reachability.git'
pod 'FreeStreamer'
end

target 'myapp-tvos' do
platform :tvos, '9.0'
shared_pods_app
end

@muhku
Copy link
Owner

muhku commented Mar 10, 2018

Official Reachability repo is still missing tvos, thus pod lint errors:

 -> FreeStreamer (3.8.3)
    - WARN  | url: There was a problem validating the URL https://github.com/muhku/FreeStreamer/.
    - ERROR | [tvOS] unknown: Encountered an unknown error (The platform of the target `App` (tvOS 9.0) is not compatible with `Reachability (3.2)`, which does not support `tvos`.) during validation.

Analyzed 1 podspec.

[!] The spec did not pass validation, due to 1 error and 1 warning.

@muhku
Copy link
Owner

muhku commented Mar 10, 2018

If there's any reasonable replacement for Reachability, I could consider, it looks like it is practically unmaintained.

@djnews24
Copy link
Contributor Author

Sure but it’s a swift framework.
https://github.com/ashleymills/Reachability.swift

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

Successfully merging this pull request may close these issues.

2 participants