2011/12/27 Måns Rullgård <[email protected]>:
> Reinhard Tartler <[email protected]> writes:
>
>> This fixes a build failure on kFreeBSD, which has recently started to
>> provide v4l2 headers. However, libavdevice/v4l2.c additionally includes
>> the header files <asm/types.h>, which is too Linux specific here.
>> Therefore, this patch restricts libavdevice's support to Linux hosts
>> until libavdevice gets fixed.
>> ---
>>  configure |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 9b510de..158fd78 100755
>> --- a/configure
>> +++ b/configure
>> @@ -2987,8 +2987,8 @@ fi
>>  texi2html -version > /dev/null 2>&1 && enable texi2html || disable texi2html
>>
>>  check_header linux/fb.h
>> -check_header linux/videodev.h
>> -check_header linux/videodev2.h
>> +test $target_os = 'linux' && check_header linux/videodev.h
>> +test $target_os = 'linux' && check_header linux/videodev2.h
>>  check_header sys/videoio.h
>
> This was solved differently, right?  Either way, this patch is not OK.

yes, the right fix is to make avdevice/v4l2.c less linux specifc.

-- 
regards,
    Reinhard
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to