script.module.youtube.dl-2021.06.06+matrix.zip, fehler bei download, mit patch

MarkusE

Hallo,

mit "script.module.youtube.dl-2021.06.06+matrix.zip" funktioniert der Download nicht mehr (das Anschauen geht noch).
Liegt wohl an der Umstellung zu Python 3.

Der attachte Patch behebt diesen Fehler.

~Markus

PvD

Das reiche ich mal an @Publish3r weiter...

Publish3r

Danke @MarkusE.

Update folgt gleich.

PvD

@Publish3r, ich habe bereits gepatcht, geht mit meiner IDE (PyCharm) recht einfach. Ich kann's auch hochladen.

Publish3r

Habs gerade fertig. Mir ists egal.

PvD

Dann lade Du das hoch.

EDIT: Schon gesehen, erledigt

MarkusE

Ihr seid ja super schnell

MarkusE

Hi,

Noch eine Änderung in script.module.youtube.dl-2021.12.17+matrix, in der Datei
.kodi/addons/http://script.module.youtube.dl/service.py , bitte

"t.isAlive()" ersetzen durch "t.is_alive()" (in Zeile 60).

Diese Änderung wird für neuere Python Versionen benötigt.

PvD

Ist im Repo. script.module.youtube.dl-2021.12.17+matrix.1

Publish3r

@PvD hast du das gefixt? Frage, weil ein neuer Upload im Repo ist.

Und allgemein muss man sich für die Zukunft sicher eh was anderes überlegen. Das Addon bzw. Youtube.DL scheint ja nicht weiter entwickelt zu werden und es gibt glaube ich schon nen anderes Ausweichprojekt.

PvD

Jupp, hab' ich. Auf deinem Github ist das Addon nicht zu finden, sonst hätte ich einen PR gemacht.

Zitat von Publish3r

Und allgemein muss man sich für die Zukunft sicher eh was anderes überlegen. Das Addon bzw. Youtube.DL scheint ja nicht weiter entwickelt zu werden

Zumindest in den offiziellen Repos für Matrix und Nexus ist es drin, aber k.A. wie aktuell das Addon nun wirklich ist:

http://mirrors.kodi.tv/addons/matrix/…ule.youtube.dl/
http://mirrors.kodi.tv/addons/nexus/s…ule.youtube.dl/

Publish3r

Am Addon selber muss ja nicht viel gemacht werden, eigentlich gar nichts, außer alle Jubeljahre mal.
Man muss ja nur immer den aktuellen Inhalt von https://github.com/ytdl-org/youtu…ster/youtube_dl runterladen und im Addon einfügen.
Dort gibt es aber seit Ende 2021 wohl nur noch Community PRs.

MarkusE

Noch einer:
in .kodi/addons/http://script.module.youtube.dl/lib/main.py , Zeile 135:

"xbmcgui.Dialog().ok(T(32045), line1, '', line2)"

ersetzen durch:
"xbmcgui.Dialog().ok(T(32045), line1 + '\n' + line2)"

> es gibt glaube ich schon nen anderes Ausweichprojekt.
Welches Projekt ist das?

Publish3r

Zitat von MarkusE

in .kodi/addons/http://script.module.youtube.dl/lib/main.py , Zeile 135:
"xbmcgui.Dialog().ok(T(32045), line1, '', line2)"
ersetzen durch:
"xbmcgui.Dialog().ok(T(32045), line1 + '\n' + line2)"

Erledigt.

Zitat von MarkusE

Welches Projekt ist das?

https://github.com/yt-dlp/yt-dlp

PELK

Scheint, dass die Problem durch Fehler beim Import von Submodulen entstehen. Ich habe die entsprechenden Zeilen angepasst. Bei einem Dialog in YDStreamUtils hat sich die Parameterliste geändert. Außerdem ist auch die deutsche Übersetzung etwas verschönert worden. Bis auf das Management der Queue läufts damit bei mit unter Windows 10/Kodi 19.4 wieder.

Code

File difference report generated by CSDiff by ComponentSoftware on 11.11.2022 17:13


Base file: C:\Users\Peter\Desktop\YDStreamExtractor (2022.05.23).py
Compared file: C:\Users\Peter\Desktop\YDStreamExtractor (patched).py


6c6
< import YoutubeDLWrapper
---
> from lib import YoutubeDLWrapper
8,9c8,9
< import YDStreamUtils as StreamUtils
< from yd_private_libs import util, servicecontrol
---
> from lib import YDStreamUtils as StreamUtils
> from lib.yd_private_libs import util, servicecontrol




<---------------------  End of report  --------------------->

Alles anzeigen

Code

File difference report generated by CSDiff by ComponentSoftware on 11.11.2022 17:15


Base file: C:\Users\Peter\Desktop\YDStreamUtils (2022.05.23).py
Compared file: C:\Users\Peter\Desktop\YDStreamUtils (patched).py


5c5
< from yd_private_libs import util
---
> from lib.yd_private_libs import util
210c210
<             new = xbmcgui.Dialog().yesno(T(32005), T(32006), path, T(32007), T(32008), T(32009))
---
>             new = xbmcgui.Dialog().yesno(T(32005), T(32006) + "\n" + path + "\n" + T(32007), T(32008), T(32009))




<---------------------  End of report  --------------------->

Alles anzeigen

Code

File difference report generated by CSDiff by ComponentSoftware on 11.11.2022 17:16


Base file: C:\Users\Peter\Desktop\YoutubeDLWrapper (2022.05.23).py
Compared file: C:\Users\Peter\Desktop\YoutubeDLWrapper (patched).py


6,7c6,7
< from yd_private_libs import util, updater
< import YDStreamUtils as StreamUtils
---
> from lib.yd_private_libs import util, updater
> from lib import YDStreamUtils as StreamUtils
13c13
< from youtube_dl.utils import std_headers, DownloadError  # noqa E402
---
> from lib.youtube_dl.utils import std_headers, DownloadError  # noqa E402
36c36
<     from yd_private_libs import _subprocess
---
>     from lib.yd_private_libs import _subprocess
41c41
<     import youtube_dl
---
>     from lib import youtube_dl
342c342
<     from youtube_dl import downloader
---
>     from lib.youtube_dl import downloader




<---------------------  End of report  --------------------->

Alles anzeigen

Im Anhang sind die gepatchten Module. Leider habe ich so gut wie keine Ahnung von Python und dem Import-System. Ich vermute daher, dass es womöglich eine viel elegantere Methode gibt, die fehlende Auflösung der korrekten Moduladressen herzustellen. Ich werde jetzt testen, ob es auch auf Libreelec funktioniert.

PELK

Mit weiteren Patches in servicecontrol.py und script-module-youtube-dl-options_dialog.xml funktioniert die Queue-Verwaltung auch. Hier habe ich auch noch die Bestätigungs-Dialoge lokalisiert (nur Detusch und Englisch), daher noch mal neue String.po's.
Auf Libreelec läufts auch, wenn man den DASH-Suport im Youtube-Addon (nicht in Youtube.dl!) abschaltet, sonst hören die Downloads sofort wieder auf (war auf dem PC so eingestellt, ohne das ich's wusste). Die Einstellung DASH in youtube.dl hat hierbei leider keine Wirkung. Vorsicht bei der Aufnahme von Live-Übertragungen: Diese lassen sich in Libreelec nur durch Neustart von Kodi abbrechen. Auf dem PC funktioniert die Live-Aufnahme gar nicht.

Im Anhang nochmal alle geänderten Dateien in einem Zip mit der Addon-Verzeichnisstruktur.

Code

File difference report generated by CSDiff by ComponentSoftware on 12.11.2022 15:36


Base file: C:\Users\Peter\Desktop\servicecontrol (2022.05.23+matrix.1).py
Compared file: C:\Users\Peter\Desktop\servicecontrol (patched).py


10a11,12
> T = util.T # See enhanced localization files (only English and German)
> 
52,53c54,55
<                 return xbmcgui.Dialog().ok('Queue Empty', 'No downloads are in the queue.')
<             d = util.xbmcDialogSelect('Select Item To Delete')
---
>                 return xbmcgui.Dialog().ok(T(32046), T(32047)) # See enhanced localization files (only English and German)
>             d = util.xbmcDialogSelect(T(32048)) # See enhanced localization files (only English and German)
55c57
<                 data = json.loads(binascii.unhexlify(val))['data']
---
>                 data = json.loads(val)['data']




<---------------------  End of report  --------------------->

Alles anzeigen

Code

File difference report generated by CSDiff by ComponentSoftware on 12.11.2022 15:44


Base file: C:\Users\Peter\Desktop\script-module-youtube-dl-options_dialog (2022.05.23+matrix.1).xml
Compared file: C:\Users\Peter\Desktop\script-module-youtube-dl-options_dialog (patched).xml


68c68
<                 <enable>String.IsEqual(Window(10000).Property(script.module.youtube.dl_STATUS),ACTIVE) + IntegerGreaterThan(Window(10000).property(XBMCJsonRAFifoQueue_download.queue.size),0)</enable>
---
>                 <enable>String.IsEqual(Window(10000).Property(script.module.youtube.dl_STATUS),ACTIVE) + Integer.IsGreater(Window(10000).property(XBMCJsonRAFifoQueue_download.queue.size),0)</enable>




<---------------------  End of report  --------------------->

Alles anzeigen

Jetzt mitmachen!