script.module.youtube.dl icon indicating copy to clipboard operation
script.module.youtube.dl copied to clipboard

Make script Python 3 compatible

Open goggle opened this issue 6 years ago • 3 comments

I'm currently working on making my Kodi plugins to work with Python 3 (see https://kodi.tv/article/attention-addon-developers-migration-python-3). It is suggested to make all the plugins/scripts compatible with both, Python 2 and Python 3, so that the transition will be smooth.

Currently, script.module.youtube.dl does not work with Python 3. On a Python 3 Kodi test build, I get the following error, when I try to import it:

script.module.youtube.dl/lib/YDStreamExtractor.py", line 418
                                                except YoutubeDLWrapper.youtube_dl.DownloadError, e:
                                                                                                ^
                                            SyntaxError: invalid syntax

There are probably many more lines, where the code is not compatible to Python 3.

Is it planned to release script.module.youtube.dl with Python 3 support?

goggle avatar Mar 02 '19 04:03 goggle

Hi @ruuk, Are you interested on a PR of your add-on for Python 2 AND 3 compatibility? I am currently working on it.

sy6sy2 avatar Oct 01 '19 12:10 sy6sy2

A PR would be great!

ruuk avatar Oct 01 '19 17:10 ruuk

A first attempt here https://github.com/ruuk/script.module.youtube.dl/pull/52

sy6sy2 avatar Feb 09 '20 11:02 sy6sy2