[FFmpeg-devel] [PATCH][8/8] Add VA API deps to configure

[Diego Biurrun]

On Mon, Feb 09, 2009 at 01:07:15PM +0100, Gwenole Beauchesne wrote:
> 
> On Mon, 9 Feb 2009, Diego Biurrun wrote:
> 
> >> +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?
> 
> In the upstream implementation? No, but in the internal tree at Intel and 
> in our packages, yes. How do you want to fetch CFLAGS and the necessary 
> libs otherwise? Note: VA API headers are now located in their own 
> directory, so a -I/some/path is needed.

That sounds like bad design similar to the nonsense that the dirac people
built.  Why can't you simply use

#include <some/path/whatever.h>

?

Diego