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

[Diego Biurrun]

On Tue, Mar 03, 2009 at 11:26:07AM +0100, Gwenole Beauchesne wrote:
> 
> 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.

I think you should be able to get rid of the vaapi option altogether,
like I did for xvmc.  Some suggestions below:

> --- configure	(revision 17752)
> +++ configure	(working copy)
> @@ -1055,6 +1064,7 @@
>  wmav2_encoder_select="fft mdct"
>  wmv1_encoder_select="aandct"
>  wmv2_encoder_select="aandct"
> +wmv3_vaapi_decoder_deps="vaapi"
>  wmv3_vdpau_decoder_deps="vdpau"
>  zlib_decoder_select="zlib"
>  zlib_encoder_select="zlib"
> @@ -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"

You could make all the decoders depend on va_va_h directly.

> @@ -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

Is this really needed?

Diego