Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

Am 05.03.2018 um 21:17 schrieb Lou Logan: On Mon, Mar 5, 2018, at 11:07 AM, Timo Rothenpieler wrote: I tried to do that multiple times already, but trac won't let me login, it just eventually runs into a Gateway Timeout if I try. Weird. I can't duplicate that issue. Is anyone else

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Mon, Mar 5, 2018, at 11:07 AM, Timo Rothenpieler wrote: > > I tried to do that multiple times already, but trac won't let me login, > it just eventually runs into a Gateway Timeout if I try. Weird. I can't duplicate that issue. Is anyone else experiencing this? If you give me the info I can

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

Am 05.03.2018 um 20:46 schrieb Lou Logan: On Tue, Feb 27, 2018, at 6:24 AM, Timo Rothenpieler wrote: Patch applied. Could you mention the new repo, and perhaps some simple instructions, on HWAccelIntro? I've seen a few confused users. I would but I'm not a HW user and I don't want to add

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, Feb 27, 2018, at 6:24 AM, Timo Rothenpieler wrote: > > Patch applied. Could you mention the new repo, and perhaps some simple instructions, on HWAccelIntro? I've seen a few confused users. I would but I'm not a HW user and I don't want to add anything incorrect.

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

+cuda_deps="ffnvcodec"> cuda_deps_any="libdl LoadLibrary" cuvid_deps="cuda" d3d11va_deps="dxva_h ID3D11VideoDecoder ID3D11VideoContext" @@ -5811,6 +5812,9 @@ check_type "va/va.h va/va_enc_vp9.h" "VAEncPictureParameterBufferVP9" check_type "vdpau/vdpau.h" "VdpPictureInfoHEVC"

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On 2/25/2018 9:20 PM, Marton Balint wrote: > From: Timo Rothenpieler > > External headers are no longer welcome in the ffmpeg codebase because they > increase the maintenance burden. However, in the NVidia case the vanilla > headers need some modifications to be usable in

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, 27 Feb 2018 12:27:10 +0100 Hendrik Leppkes wrote: > On Tue, Feb 27, 2018 at 12:13 PM, Nicolas George wrote: > > Hendrik Leppkes (2018-02-27): > >> Something works today, you want to refactor it, you get to keep it > >> working. Its a simple rule.

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

Hendrik Leppkes (2018-02-27): > Your entire approach revolves around it being an obscure feature that > noone is going to miss, so you can take that as an objection. > > The original author never submitted his patch because he didn't finish > it yet. Someone else finds it somewhere and sends it

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, Feb 27, 2018 at 12:29 PM, Nicolas George wrote: > Hendrik Leppkes (2018-02-27): >> I know you Linux people don't want to hear it, but Windows is not an >> "obscure" platform. > > Maybe. But I have to emphasize that you refused to give comment to the > practical part of my

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

Hendrik Leppkes (2018-02-27): > I know you Linux people don't want to hear it, but Windows is not an > "obscure" platform. Maybe. But I have to emphasize that you refused to give comment to the practical part of my message. Does it means you do not have an objection? We can go ahead, then.

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, Feb 27, 2018 at 12:13 PM, Nicolas George wrote: > Hendrik Leppkes (2018-02-27): >> Something works today, you want to refactor it, you get to keep it >> working. Its a simple rule. > > It is a simple rule, but not one that must be followed blindly. Please > do not be

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, 27 Feb 2018 12:01:48 +0100 Hendrik Leppkes wrote: > On Tue, Feb 27, 2018 at 11:59 AM, wm4 wrote: > > On Tue, 27 Feb 2018 11:49:25 +0100 > > Hendrik Leppkes wrote: > > > >> On Tue, Feb 27, 2018 at 11:45 AM, wm4

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

Hendrik Leppkes (2018-02-27): > Something works today, you want to refactor it, you get to keep it > working. Its a simple rule. It is a simple rule, but not one that must be followed blindly. Please do not be dogmatic. Sometimes, we have to "break" an obscure case in order to make progress. We

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, Feb 27, 2018 at 11:59 AM, wm4 wrote: > On Tue, 27 Feb 2018 11:49:25 +0100 > Hendrik Leppkes wrote: > >> On Tue, Feb 27, 2018 at 11:45 AM, wm4 wrote: >> > On Mon, 26 Feb 2018 22:15:04 +0100 (CET) >> > Marton Balint

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, 27 Feb 2018 11:49:25 +0100 Hendrik Leppkes wrote: > On Tue, Feb 27, 2018 at 11:45 AM, wm4 wrote: > > On Mon, 26 Feb 2018 22:15:04 +0100 (CET) > > Marton Balint wrote: > > > >> On Mon, 26 Feb 2018, Timo Rothenpieler wrote: >

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Tue, Feb 27, 2018 at 11:45 AM, wm4 wrote: > On Mon, 26 Feb 2018 22:15:04 +0100 (CET) > Marton Balint wrote: > >> On Mon, 26 Feb 2018, Timo Rothenpieler wrote: >> >> > The reason I haven't pushed this yet is because it breaks the build on >> > MSVC, and I

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Mon, 26 Feb 2018 22:15:04 +0100 (CET) Marton Balint wrote: > On Mon, 26 Feb 2018, Timo Rothenpieler wrote: > > > The reason I haven't pushed this yet is because it breaks the build on > > MSVC, and I haven't gotten around to figuring out pkg-config for it. > > Right. I

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

On Mon, 26 Feb 2018, Timo Rothenpieler wrote: The reason I haven't pushed this yet is because it breaks the build on MSVC, and I haven't gotten around to figuring out pkg-config for it. Right. I picked this up because I thought it would be nice if we could go through with it before the

Re: [FFmpeg-devel] [PATCH] compat: remove in-tree NVidia headers

The reason I haven't pushed this yet is because it breaks the build on MSVC, and I haven't gotten around to figuring out pkg-config for it. smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org