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

Carl Eugen Hoyos ceffmpeg at gmail.com
Mon Dec 4 22:18:10 EET 2017


2017-12-04 21:12 GMT+01:00 Timo Rothenpieler <timo at rothenpieler.org>:
> Am 04.12.2017 um 20:58 schrieb Carl Eugen Hoyos:
>>
>> 2017-12-04 20:41 GMT+01:00 Timo Rothenpieler <timo at rothenpieler.org>:
>>>
>>> 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


More information about the ffmpeg-devel mailing list