[MPlayer-cvslog] r34132 - trunk/configure

[Diego Biurrun]

On Sun, Jan 08, 2012 at 01:56:50PM +0100, Reinhard Tartler wrote:
> On So, Sep 25, 2011 at 15:28:25 (CEST), reimar wrote:
> >
> > Log:
> > Set up HAVE_VFP_ARGS and CONFIG_THUMB correctly to fix compilation for ARM.
> >
> > Modified:
> >    trunk/configure
> >
> > Modified: trunk/configure
> > ==============================================================================
> 
> [...]
> 
> > @@ -2872,6 +2895,19 @@ if arm ; then
> >  
> > +  echocheck "ARM THUMB"
> > +  if test $armthumb = "auto" ; then
> > +    armthumb=no
> > +  fi
> > +  if test $armthumb = "yes" ; then
> > +    extra_cflags="$extra_cflags -mthumb"
> > +    def_armthumb='#define CONFIG_THUMB 1'
> > +  else
> > +    extra_cflags="$extra_cflags -marm"
> > +    def_armthumb='#define CONFIG_THUMB 0'
> > +  fi
> 
> I think -marm is wrong:
> 
> https://buildd.debian.org/status/fetch.php?pkg=mplayer&arch=armel&ver=2%3A1.0~rc4.dfsg1%2Bsvn34492-1&stamp=1325969800
> 
> The build fails because the libmpeg2 uses armv5 speicific
> instruction.

I think we should finally throw out the forked copy of libmpeg2.

Diego