[FFmpeg-devel] [PATCH] Don't print vdpau upgrade notice on platforms without vdpau

Jai Menon jmenon86
Mon Jan 4 16:07:55 CET 2010


On Mon, Jan 04, 2010 at 03:59:33PM +0100, Diego Biurrun wrote:
> On Mon, Jan 04, 2010 at 08:17:27PM +0530, Jai Menon wrote:
> > On Mon, Jan 04, 2010 at 03:33:01PM +0100, Diego Biurrun wrote:
> > > On Mon, Jan 04, 2010 at 07:52:34PM +0530, Jai Menon wrote:
> > > > 
> > > > As in subject. Currently, configure insists on printing the message on
> > > > all platforms irrespective of whether the headers are installed or
> > > > not.
> > > 
> > > Thanks, this is very annoying..
> > > 
> > > > --- a/configure
> > > > +++ b/configure
> > > > @@ -2500,10 +2500,12 @@ check_func XOpenDisplay -lX11           &&
> > > >  
> > > > +if enabled vdpau_vdpau_h; then
> > > >  check_cpp_condition \
> > > 
> > > enabled vdpau_vdpau_h &&
> > 
> > right, that works too. okay to apply that change?
> 
> Yes, I would have objected otherwise.  I generally think that you should
> just commit such changes directly if you take the reviewer comments into
> account.

I'll keep that in mind for trivial future patches :)

Thanks, applied.

-- 
Jai Menon




More information about the ffmpeg-devel mailing list