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

Error compiling for tvOS - deprecated functions in lib/cdm/cdm/base/native_library_mac.mm #1590

Open
WugtheFlunk opened this issue Jun 27, 2024 · 3 comments

Comments

@WugtheFlunk
Copy link

Describe the problem

LoadNativeLibrary function on line 45 has portion of code on line 77:

native_lib->bundle_resource_ref` = CFBundleOpenBundleResourceMap(bundle);

Similarly, UnloadNativeLibrary function on line 83 has deprecated function on lines 88 & 89:

      CFBundleCloseBundleResourceMap(library->bundle,
                                     library->bundle_resource_ref);

These functions are not available in tvOS and they are deprecated in macOS as of version 10.15.

More info here: https://developer.apple.com/documentation/corefoundation/1537133-cfbundleclosebundleresourcemap?language=objc

Possible fix

No response

Steps to reproduce

No response

Debug log

No Kodi log because this is a compilation issue.

Stream manifest file(s)

No response

Additional info

No response

Operating system(s)

Apple (tvOS)

Operating system version(s)

No response

InputStream Adaptive version(s)

21.4.10

Kodi version(s)

N/A - Build issue.

@WugtheFlunk WugtheFlunk added the Triage: Needed (managed by bot) issue that was just created and needs someone looking at it label Jun 27, 2024
@CastagnaIT
Copy link
Collaborator

i made a PR #1593 but idk if changes are right
because idk apple enviroment and its needed test these changes

are you able to build and test it on osx and tvos?
if so please read "How has this been tested" section on PR

@CastagnaIT
Copy link
Collaborator

now that i think about it better my pr its not full correct
because yes it check at build condition, but not in a running enviroment

@CastagnaIT
Copy link
Collaborator

CastagnaIT commented Aug 19, 2024

can you post the complete sequence of commands that you use to build ISA?

i was looking again, but on our CI buildings i dont see this problem on tvOS
the lib/cdm folder should not be built because the system (darwin_embedded) is excluded

if(NOT CORE_SYSTEM_NAME STREQUAL ios AND NOT CORE_SYSTEM_NAME STREQUAL darwin_embedded)

so my suspect is that you are using a different way to build it

@CastagnaIT CastagnaIT removed the Triage: Needed (managed by bot) issue that was just created and needs someone looking at it label Aug 19, 2024
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 a pull request may close this issue.

2 participants