On Mon, Apr 04, 2011 at 00:27:05 (CEST), Måns Rullgård wrote: > Luca Barbato <[email protected]> writes: > >> On 04/04/2011 12:16 AM, Måns Rullgård wrote: >>> Luca Barbato <[email protected]> writes: >>> >>>> Make the behaviour consistent with the other external deps. >>>> --- >>>> configure | 4 +++- >>>> 1 files changed, 3 insertions(+), 1 deletions(-) >>>> >>>> diff --git a/configure b/configure >>>> index fab4f2b..0aae67c 100755 >>>> --- a/configure >>>> +++ b/configure >>>> @@ -2827,7 +2827,9 @@ for thread in $THREADS_LIST; do >>>> done >>>> >>>> check_lib math.h sin -lm >>>> -disabled vaapi || check_lib va/va.h vaInitialize -lva >>>> +enabled vaapi && require vaapi va/va.h vaInitialize -lva && >>>> + { check_cpp_condition va/va_version.h >>>> "VA_CHECK_VERSION(0,32,0)" || >>>> + die "Please upgrade to VA-API >= 0.32"; } >>> >>> You are changing the test to require a specific version. Why? >> >> The patchset just warns if the version is incorrect I'd rather require >> the proper version. > > I'm talking about this very patch. Current git master only checks for > the library existence. Since when do we require 0.32? It is still > marked unstable in Gentoo, and other distributions are usually slower.
We shouldn't require 0.32, bug make use of it if it is available. The necessary code has just been posted to this list. -- Gruesse/greetings, Reinhard Tartler, KeyID 945348A4 _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
