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

Diego Biurrun diego
Mon Jan 4 15:59:33 CET 2010


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.

Diego



More information about the ffmpeg-devel mailing list