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

[Steve Lhomme]

OK. My understanding was that this would add the feature to anyone, but 
it's only for people who already have youtube-dl.

On 2020-09-21 12:07, Rémi Denis-Courmont wrote:
> Hi,
> 
> To the extent that we use Python to build VLC, and that it does not come 
> with the OS, then we may need to provide Python for the build system. So 
> that depends on the build OS really. But that has nothing to do 
> whatsoever with this patch set, which does not introduce any build-time 
> dependency on Python and does not cause Python to be shipped with VLC.
> 
> As for YoutubeDL, we don't use it to build VLC and we don't ship it in 
> any form.
> 
> This is completely different from, say, OpenSSL, where you'd end up both 
> needing the headers and import library to build, and where you'd ship 
> byte code in whichever VLC module linked directly or indirectly with 
> OpenSSL.
> 
> Le 21 septembre 2020 12:50:15 GMT+03:00, Steve Lhomme <robux4 at ycbcr.xyz> 
> a écrit :
> 
>     On 2020-09-21 11:47, Rémi Denis-Courmont wrote:
> 
>         Hi,
> 
>         Of course not. We don't package a compiler, a kernel, a display
>         server
>         and a PulseAudio daemon. We don't package the decompressor
>         binaries that
>         the decomp filter uses. We don't even package ALSA and GL
>         libraries that
>         we have build-time dependencies on.
> 
>         We need to provide sources for whatever we provide binaries. We
>         don't
>         provide binaries of YoutubeDL nor Python 3.
> 
> 
>     Doesn't it violate the GPL ? Python and all the things you cited can be
>     considered as part of the OS but youtube-dl I highly doubt it.
> 
>         Le 21 septembre 2020 12:39:26 GMT+03:00, Steve Lhomme
>         <robux4 at ycbcr.xyz>
>         a écrit :
> 
>         Virtual Environment
>         https://docs.python.org/3/tutorial/venv.html
> 
>         The other thing is that whatever VLC needs, it doesn't
>         pollute the
>         system environment, potentially introducing incompatible
>         dependencies.
> 
>         Using a venv does not seem very useful here.
>         If packagers want to do that, thats a different story but
>         in general I dont think VLC should set up a venv because
>         it does not even install any dependency so it seems pointless
>         to do.
>         Even if we were to package it in contribs, a venv is probably
>         not needed as it seems youtube-dl has no dependencies itself
>         so it could just be easily imported without a venv.
> 
> 
>         I think we need to package it in VLC since we need to be able to
>         make
>         package sources of whatever we use in VLC, our code, the
>         contribs and
>         the tools.
>         ------------------------------------------------------------------------
>         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é.
>         ------------------------------------------------------------------------
>         vlc-devel mailing list
>         To unsubscribe or modify your subscription options:
>         https://mailman.videolan.org/listinfo/vlc-devel
> 
>     ------------------------------------------------------------------------
>     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é.
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
>