Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-17 Thread Timo Rothenpieler
> But before doing any of this, there has to be an agreement, whether or not > the header should be included at all. Of course. Here's what I intend to do once I find some time again, which could take a few days/weeks: Split this up into three patches. One that adds a check for NVENC Version 6,

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-15 Thread Andreas Cadhalpun
On 15.12.2015 14:42, Timo Rothenpieler wrote: > So, to get somewhere with this, would everybody be ok if I change this > to remove the non-free marking, but keep it disabled by default for now? That would be OK for me. > Or is putting that exception-text in nvenc.c enough to make enabling it >

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-15 Thread Timo Rothenpieler
So, to get somewhere with this, would everybody be ok if I change this to remove the non-free marking, but keep it disabled by default for now? Or is putting that exception-text in nvenc.c enough to make enabling it by default viable? signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-15 Thread Jean-Baptiste Kempf
On 15 Dec, Timo Rothenpieler wrote : > So, to get somewhere with this, would everybody be ok if I change this > to remove the non-free marking, but keep it disabled by default for now? I still don't understand why you want to fork this header into the FFmpeg tree. -- Jean-Baptiste Kempf

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Matt Oliver
> > When running ffmpeg on Debian (main), where these Nvidia blobs > are not present and not needed (because nouveau is used as driver), > no component of the operating system, major or not, > has been distributed with Nvidia's blobs, so the system library > exception does not apply. > > I'm not

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Philip Langdale
On 2015-12-12 20:33, Nicolas George wrote: Le duodi 22 frimaire, an CCXXIV, Philip Langdale a écrit : it would be highly illogical to conclude that section 6/7 do not apply to the original code itself and that we need to construct a separate entity that does the combination for it to be

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > I don't agree as I have already explained previously. Fortunately, your arguments have no legal standing. > Have a look at what some of the proprietary licenses require for distribution > that has absolutely nothing to do with how

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > However, the (L)GPL has a rule that allows distribution of a > binary only together with the complete corresponding source code, > that also includes the source code of any dynamically loaded > library needed at run-time. No, it does

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > Is in a previous mail. I looked and did not see it. > I'm not continuing this silly conversation. > You can look up the meaning of words in a dictionary. This has nothing silly. Having the code of a codec interpret the bitstream of

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 13:27, Philip Langdale wrote: > On 2015-12-12 19:26, Andreas Cadhalpun wrote: >> Anyway, I think your idea about a license exception would be a good solution. >> We could add a special exception to nvenc.c, allowing its use with Nvidia's >> blobs, something like [1]: >> >> In

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 12:55, Nicolas George wrote: > Le primidi 21 frimaire, an CCXXIV, Carl Eugen Hoyos a écrit : >> I am glad we agree that there is no difference (license-wise) if >> a library is linked statically, dynamically or via dynamic >> loading;-) > > You are, unfortunately, completely

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 10:37, Matt Oliver wrote: >> >> When running ffmpeg on Debian (main), where these Nvidia blobs >> are not present and not needed (because nouveau is used as driver), >> no component of the operating system, major or not, >> has been distributed with Nvidia's blobs, so the system

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 11:35, Hendrik Leppkes wrote: > On Sat, Dec 12, 2015 at 11:25 AM, Andreas Cadhalpun > wrote: >> On 12.12.2015 10:50, Hendrik Leppkes wrote: >>> We should just add an exception into the license to explicitly allow >>> using it with the NVIDIA CUDA

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le primidi 21 frimaire, an CCXXIV, Carl Eugen Hoyos a écrit : > I am glad we agree that there is no difference (license-wise) if > a library is linked statically, dynamically or via dynamic > loading;-) You are, unfortunately, completely wrong. It makes all the difference in the world. Static

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Philip Langdale
On 2015-12-12 19:26, Andreas Cadhalpun wrote: On 12.12.2015 11:35, Hendrik Leppkes wrote: On Sat, Dec 12, 2015 at 11:25 AM, Andreas Cadhalpun wrote: On 12.12.2015 10:50, Hendrik Leppkes wrote: We should just add an exception into the license to explicitly

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > It does. [citation needed] > Multimedia files can't be executed, only decoded. Define the difference. Regards, -- Nicolas George signature.asc Description: Digital signature ___

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 10:50, Hendrik Leppkes wrote: > On Sat, Dec 12, 2015 at 10:39 AM, Andreas Cadhalpun > wrote: >> On 12.12.2015 01:46, Philip Langdale wrote: >>> On 2015-12-12 00:03, Andreas Cadhalpun wrote: On 11.12.2015 09:41, Carl Eugen Hoyos wrote: > My

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 14:42, Nicolas George wrote: > Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> It does. > > [citation needed] Is in a previous mail. >> Multimedia files can't be executed, only decoded. > > Define the difference. I'm not continuing this silly conversation. You

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 01:46, Philip Langdale wrote: > On 2015-12-12 00:03, Andreas Cadhalpun wrote: >> On 11.12.2015 09:41, Carl Eugen Hoyos wrote: >>> My point is that so far several people have said that if nvenc >>> is a system library there is no issue (and I fully agree). I >>> didn't see a mail (and

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Hendrik Leppkes
On Sat, Dec 12, 2015 at 10:39 AM, Andreas Cadhalpun wrote: > On 12.12.2015 01:46, Philip Langdale wrote: >> On 2015-12-12 00:03, Andreas Cadhalpun wrote: >>> On 11.12.2015 09:41, Carl Eugen Hoyos wrote: My point is that so far several people have said that

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Philip Langdale a écrit : > it would be highly illogical to conclude that section 6/7 do not apply to > the original code itself and that we need to construct a separate entity > that does the combination for it to be licence compliant. It would be fairly

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 14:29, Nicolas George wrote: > Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> However, the (L)GPL has a rule that allows distribution of a >> binary only together with the complete corresponding source code, >> that also includes the source code of any dynamically

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Hendrik Leppkes
On Sat, Dec 12, 2015 at 11:25 AM, Andreas Cadhalpun wrote: > On 12.12.2015 10:50, Hendrik Leppkes wrote: >> On Sat, Dec 12, 2015 at 10:39 AM, Andreas Cadhalpun >> wrote: >>> On 12.12.2015 01:46, Philip Langdale wrote: On

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 14:03, Nicolas George wrote: > Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> I don't agree as I have already explained previously. > > Fortunately, your arguments have no legal standing. Neither have yours. > The (L)GPL does not have a rule against distributing a

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-11 Thread Andreas Cadhalpun
On 11.12.2015 09:41, Carl Eugen Hoyos wrote: > On Friday 11 December 2015 12:16:48 am Andreas Cadhalpun wrote: >> Well, the problem is that the answer may depend on the system. > > If Nvidia offers Graphics Driver for download on its ftp server > for multiple operating systems, they are either

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-11 Thread Philip Langdale
On 2015-12-12 00:03, Andreas Cadhalpun wrote: On 11.12.2015 09:41, Carl Eugen Hoyos wrote: On Friday 11 December 2015 12:16:48 am Andreas Cadhalpun wrote: Well, the problem is that the answer may depend on the system. If Nvidia offers Graphics Driver for download on its ftp server for

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-11 Thread Carl Eugen Hoyos
On Friday 11 December 2015 12:16:48 am Andreas Cadhalpun wrote: > On 11.12.2015 00:03, Carl Eugen Hoyos wrote: > > On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: > >> @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab > >> > >> die_license_disabled nonfree libaacplus > >>

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Hendrik Leppkes
On Fri, Dec 11, 2015 at 12:03 AM, Carl Eugen Hoyos wrote: > On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: >> @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab >> >> die_license_disabled nonfree libaacplus >> die_license_disabled nonfree libfaac >>

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Philip Langdale
On 2015-12-08 02:53, Timo Rothenpieler wrote: Nvidia finaly decided to put a propper MIT license on their nvenc header, so it can be included, removing any external dependencies for nvenc and making it no longer require the non-free flag. nvenc.h is the original nvEncodeApi.h from the NVENC

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Carl Eugen Hoyos
On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: > @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab > > die_license_disabled nonfree libaacplus > die_license_disabled nonfree libfaac > -die_license_disabled nonfree nvenc Sorry, but this makes absolutely no sense imo: I

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-10 Thread Andreas Cadhalpun
On 11.12.2015 00:03, Carl Eugen Hoyos wrote: > On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: >> @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab >> >> die_license_disabled nonfree libaacplus >> die_license_disabled nonfree libfaac >> -die_license_disabled nonfree nvenc >