[vlc-devel] [PATCH 1/2] share: add wrapper script for the YoutubeDL Python module

[Rémi Denis-Courmont]

Hi,

1) Yes, I meant JSON, sorry. And we do have a JSON parser, but it makes casts that look very dubious looking to me. And we'd still need to write a parser for the proprietary JSON schema. I'd rather reuse an existing parser in VLC.

I don't see what benefits there would be to using the official YoutubeDL script instead of a custom one. Either way it's a Python script that uses the official YoutubeDL module API.

2) YoutubeDL can do lots of things, like downloading or converting the video using FFmpeg. It even has some adaptive streaming support. However, they are targeted at offline downloads, so I think VLC's own implementation is actually better.

Besides, I'm not interested in any of those features that VLC already has, unlike the hundreds of site handlers that YoutubeDL has and VLC doesn't. Maybe someone wants to import more features from YoutubeDL, but that's just not in scope of this patchset.

3) I have a patch for redirect instead of single item playlist, which I purposefully did not submit. It's broken due to recursion into YoutubeDL. It's not as straight forward as for Icecast.

It's fixable: just set ytdl=false on the stream object. Not sure if people will agree to that though.

Le 21 septembre 2020 03:20:38 GMT+03:00, Pierre Ynard via vlc-devel <vlc-devel at videolan.org> a écrit :
>_______________________________________________
>vlc-devel mailing list
>To unsubscribe or modify your subscription options:
>https://mailman.videolan.org/listinfo/vlc-devel

-- 
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20200921/68b62a96/attachment.html>