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

replace deprecated function listItem.setInfo() #101

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

firsttris
Copy link
Owner

should fix:
#92

@nullket
Copy link
Collaborator

nullket commented Dec 9, 2023

I am currently not able to test the changes but based on quick research this will break support for everything before Kodi 20. But I am not 100 % certain, somebody else might want to clarify this. For me this would be welcomed as it would allow us to get rid of the python2 compatibility quirks (no second repo, using python3 features etc.) - but I am not the only one and there still might be people on 18/19. Just wanted to let you know :)

@firsttris
Copy link
Owner Author

for me its okay to move on now and drop python2.

@nullket
Copy link
Collaborator

nullket commented Dec 14, 2023

Makes the most sense for me as well but maybe another PR should first get rid of the workflow etc. for the python2 repo? In that case the lost soul still using python2 could at least use the latest published addon until the resolver breaks due to website changes. Otherwise this pr would cause an update an break the addon for python2 immediately.

@anohren
Copy link
Collaborator

anohren commented Jun 26, 2024

I'm that lost soul

Kodi 17.6 on Debian 10 on an 32 bit Intel Atom netbook with OpenGL 1.4 support. The perfect kitchen youtube player 🙂

What can I say? It's cute. This and ScummVM are its last hope (I even built Kodi 17.6 for Debian 10 for it). I guess I'll have to start hooking into the system's youtube-dl...

@anohren
Copy link
Collaborator

anohren commented Jun 30, 2024

I wasn't going to bother suggesting this just for my old netbook, but I just noticed that all 32 bit iOS devices are stuck with Kodi 18. This includes iPads with retina screens.

If the goal is to simplify development (how much development is planned?) then maybe this repo can be forked into a Python 2 source repo that just runs automated updates of youtube-dl and nothing else? That'd preserve a strong use case of repurposing old iPads -- tablets that seriously can't even browse the web anymore because of outdated browsers and certificates. I think the convenience of sending videos to iPads would be a good way to keep them from becoming planned e-waste.

That wouldn't complicate any future development, would it?

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.

3 participants