Re: [FFmpeg-devel] [PATCH 1/4] configure: require external ffnvcodec headers

2017-12-05 Thread Michael Niedermayer 
On Mon, Dec 04, 2017 at 08:41:42PM +0100, Timo Rothenpieler wrote:
> The external headers can be found at
> https://git.videolan.org/?p=ffmpeg/nv-codec-headers.git
> ---
>  configure | 6 +-
>  1 file changed, 5 insertions(+), 1 deletion(-)

i cloned the repo referenced and did a make install
but

./configure --enable-nvenc
ERROR: nvenc requested, but not all dependencies are satisfied: cuda

ls -alF /usr/local/include/ffnvcodec/
ls: cannot open directory /usr/local/include/ffnvcodec/: Permission denied


[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/4] configure: require external ffnvcodec headers

2017-12-04 Thread wm4 
On Mon,  4 Dec 2017 20:41:42 +0100
Timo Rothenpieler  wrote:

> The external headers can be found at
> https://git.videolan.org/?p=ffmpeg/nv-codec-headers.git
> ---

LGTM, although I would have preferred giving the include paths a name
not specific to ffmpeg. But as I understand, the headers contain
multiple modifications anyway, and are not the same as nvidia's, so
should be ok.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/4] configure: require external ffnvcodec headers

2017-12-04 Thread Carl Eugen Hoyos 
2017-12-04 21:12 GMT+01:00 Timo Rothenpieler :
> Am 04.12.2017 um 20:58 schrieb Carl Eugen Hoyos:
>>
>> 2017-12-04 20:41 GMT+01:00 Timo Rothenpieler :
>>>
>>> The external headers can be found at
>>> https://git.videolan.org/?p=ffmpeg/nv-codec-headers.git
>>
>>
>> Afaict, 1, 2 and four should be merged.
>>
>> Could you explain (repeat) why this is necessary?
>> I suspect this will hit many users.
>>
>> Thank you, Carl Eugen
>
>
> You mean leaving the headers in tree for now, even
> though they are completely unused after 1 and 2?

Patches 1, 2 and four should be merged, patch 3 can
be a separate patch (or could be merged).

Since the first commit will hit a large number of users,
the commit message should contain some explanation.
A longer explanation in the email suggesting the change
is imo useful.

Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/4] configure: require external ffnvcodec headers

2017-12-04 Thread Timo Rothenpieler 

Am 04.12.2017 um 20:58 schrieb Carl Eugen Hoyos:

2017-12-04 20:41 GMT+01:00 Timo Rothenpieler :

The external headers can be found at
https://git.videolan.org/?p=ffmpeg/nv-codec-headers.git


Afaict, 1, 2 and four should be merged.

Could you explain (repeat) why this is necessary?
I suspect this will hit many users.

Thank you, Carl Eugen


You mean leaving the headers in tree for now, even though they are 
completely unused after 1 and 2?




smime.p7s
Description: S/MIME Cryptographic Signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/4] configure: require external ffnvcodec headers

2017-12-04 Thread Carl Eugen Hoyos 
2017-12-04 20:41 GMT+01:00 Timo Rothenpieler :
> The external headers can be found at
> https://git.videolan.org/?p=ffmpeg/nv-codec-headers.git

Afaict, 1, 2 and four should be merged.

Could you explain (repeat) why this is necessary?
I suspect this will hit many users.

Thank you, Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel