[MPlayer-cvslog] r31387 - trunk/configure

[Reimar Döffinger]

On Tue, Jun 15, 2010 at 12:30:25PM +0200, Reinhard Tartler wrote:
> On Sun, Jun 13, 2010 at 19:18:48 (CEST), diego wrote:
> 
> > Author: diego
> > Date: Sun Jun 13 19:18:48 2010
> > New Revision: 31387
> >
> > Log:
> > Add CONFIG_LIBRTMP and HAVE_ISATTY #defines to config.h for FFmpeg.
> >
> > Modified:
> >    trunk/configure
> >
> > Modified: trunk/configure
> > ==============================================================================
> > --- trunk/configure	Sun Jun 13 19:15:55 2010	(r31386)
> > +++ trunk/configure	Sun Jun 13 19:18:48 2010	(r31387)
> > @@ -9313,6 +9313,7 @@ $def_yasm
> >  #define CONFIG_FFSERVER 0
> >  #define CONFIG_GPL 1
> >  #define CONFIG_GRAY 0
> > +#define CONFIG_LIBRTMP 0
> >  #define CONFIG_LIBVORBIS 0
> >  #define CONFIG_POWERPC_PERF 0
> >  #define CONFIG_SMALL 0
> 
> This causes the following warning for every compilation unit:
> 
> config.h:519:1: warning: "CONFIG_LIBRTMP" redefined
> config.h:467:1: warning: this is the location of the previous definition
> 
> The warning seems to be correct:
> 
> $ grep CONFIG_LIBRTMP config.h
> #define CONFIG_LIBRTMP 0
> #define CONFIG_LIBRTMP 1

Yes, Diego missed the real bug, the configure check did
#undef CONFIG_LIBRTMP
when it should have been
#define CONFIG_LIBRTMP 0
Should be fixed now.