[FFmpeg-devel] [PATCH] avcodec/nvenc: Include nvEncodeAPI v7 SDK header

[Timo Rothenpieler]

On 8/29/2016 8:43 PM, James Almer wrote:
> On 8/27/2016 9:58 AM, Timo Rothenpieler wrote:
>> @@ -5996,6 +5992,22 @@ enabled vdpau && enabled xlib &&
>>      check_lib2 "vdpau/vdpau.h vdpau/vdpau_x11.h" vdp_device_create_x11 -lvdpau &&
>>      enable vdpau_x11
>>  
>> +case $target_os in
>> +    mingw32*|mingw64*|win32|win64|linux|cygwin*)
>> +        disabled nvenc || enable nvenc
>> +        ;;
>> +    *)
>> +        disable nvenc
>> +        ;;
>> +esac
>> +
>> +if enabled nvenc; then
>> +    {
>> +        echo '#include "compat/nvenc/nvEncodeAPI.h"'
>> +        echo 'int main(void) { return 0; }'
>> +    } | check_cc -I$source_path || disable nvenc
> 
> In what situation could this test fail? nvenc is only enabled if $target_os
> is one of the supported ones, and the test does nothing but compile the
> header.

Strange/broken compiler like ancient MinGW or Cygwin, or old MSVC.

> If it only supports x86 then you can just check "enabled x86" instead.

NVENC is not supported on FreeBSD or OSX for example.