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

[Alexandre Janniaux]

Hi,

On Mon, Sep 21, 2020 at 11:53:16AM +0200, Marvin Scholz wrote:
>
>
> On 21 Sep 2020, at 11:50, Steve Lhomme wrote:
>
> > 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.
>
> We do not link to either python or youtube-dl in VLC though.

This is not a layer perspective so it probably worth nothing,
but I hardly see how something you don't ship could make your
software violate the GPL.

> >
> > > 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
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel