[FFmpeg-devel] [PATCH][8/8] Add VA API deps to configure (take 2)

[Måns Rullgård]

Gwenole Beauchesne <gbeauchesne at splitted-desktop.com> writes:

> Hi,
>
> On Mon, 9 Feb 2009, Diego Biurrun wrote:
>
>> On Fri, Feb 06, 2009 at 10:42:26AM +0100, Gwenole Beauchesne wrote:
>>>
>>> Here is the configure patch for VA API support in the plain ffmpeg tree.
>>>
>>> --- configure	(revision 17015)
>>> +++ configure	(working copy)
>>> @@ -1975,6 +2000,16 @@
>>>
>>>  ##########################################
>>> +# VA API check
>>> +
>>> +pkg=libva-0.29
>>> +check_pkg_config vaapi $pkg va.h vaInitialize
>>> +if enabled vaapi; then
>>> +    add_cflags `pkg-config --cflags $pkg`
>>> +    add_extralibs `pkg-config --libs $pkg`
>>> +fi
>>
>> Do you really need pkg-config?
>
> OK, I dropped pkg-config and updated libva. Also sanitized VA API
> headers so that they build with plain -std=c99. Too bad ISO C does not
> support anonymous unions yet.

Thanks for that.

> @@ -1085,6 +1095,7 @@
>  libx264_encoder_deps="libx264"
>  libxvid_encoder_deps="libxvid"
>  mpeg4aac_decoder_deps="libfaad"
> +vaapi_deps="va_va_h"
>  vdpau_deps="vdpau_vdpau_h vdpau_vdpau_x11_h"
>  
>  # demuxers / muxers
> @@ -1947,6 +1960,7 @@
>  check_header sys/resource.h
>  check_header sys/select.h
>  check_header termios.h
> +check_header va/va.h
>  check_header vdpau/vdpau.h
>  check_header vdpau/vdpau_x11.h
>  check_header X11/extensions/XvMClib.h
> @@ -2017,6 +2031,7 @@
>                          die "ERROR: libx264 version must be >= 0.65."; }
>  enabled libxvid    && require  libxvid xvid.h xvid_global -lxvidcore
>  enabled mlib       && require  mediaLib mlib_types.h mlib_VectorSub_S16_U8_Mod -lmlib
> +enabled vaapi      && require  libva va/va.h vaInitialize -lva
>  
>  # libdc1394 check
>  if enabled libdc1394; then

With the require call the other two checks above are not needed.

-- 
M?ns Rullg?rd
mans at mansr.com