On Fri, Dec 02, 2016 at 11:24:12PM +0100, Andreas Cadhalpun wrote:
> On 02.12.2016 19:20, Michael Niedermayer wrote:
> > On Fri, Dec 02, 2016 at 12:44:41AM +0100, Andreas Cadhalpun wrote:
> >> There is already va_va_h in vaapi_deps and I'm not sure, what else
> >> you'd like to have there.
> > 
> > i see a "disable vaapi" here but that should be done by the dependency
> > or why is it done here ?
> 
> Now I see what you mean.
> 
> >> However, it seems independent of this patch.
> > 
> > yes
> > 
> > 
> > iam ok with the patch if other are and its tested
> 
> I've tested for all AUTODETECT_LIBS that enabling the when configure
> doesn't find them triggers a configure failure after the patch.
> I've also check for a few libs that all combinations of --enable,
> --disable, no configure flag and library installed or not does the
> right thing.
> 

> Do you think there is something else that should be tested?

i dont know, maybe give others a day to comment before pushing
in case someone has an idea

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Never trust a computer, one day, it may think you are the virus. -- Compn

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to