[MPlayer-dev-eng] [PATCH] new faad detection in configure

[Diego Biurrun]

James Bilotto writes:
 > On Fri, Nov 07, 2003 at 05:41:45PM +0100, Diego Biurrun wrote:
 > Content-Description: message body text
 > > James Bilotto writes:
 > >  > On Fri, Nov 07, 2003 at 02:20:49PM +0100, Diego Biurrun wrote:
 > >  > > On Fri, Nov 07, 2003 at 04:02:06AM -0800, James Bilotto wrote:
 > >  > you missed this part
 > >  > 
 > >  > +# external faad: check if it's really faad2 :)
 > >  > +echocheck "external FAAD2 (AAC) support"
 > >  > +if test "$_faad_external" != no ; then
 > >  > +  _ld_faad='-lfaad'
 > >  > +  _inc_faad="$_inc_extra"
 > >  > 
 > >  > --enable-external-faad will fail without it. can't find the header
 > > 
 > > Hmmmm
 > > 
 > >  > +  _inc_faad="$_inc_extra"
 > > 
 > > This is just a workaround.  I call
 > > 
 > >   cc_check $_inc_faad $_ld_faad -lm && _faad_external=yes
 > > 
 > > and the compile call of cc_check is this:
 > > 
 > >   ( $_cc $CFLAGS $_inc_extra $_ld_static $_ld_extra "$TMPC" -o "$TMPO" "$@" )
 > 
 > it's not a work around, it's their on purpose for the version checking
 > 
 > cc_check -I- $_inc_faad $_ld_faad -lm && "$TMPO" >> "$TMPLOG" ; then
 >          ^^^
 > from gcc's man page
 > 
 >  -I-    Any directories you specify with `-I' options before  the  `-I-'
 >               option are searched only for the case of `#include "file"'; they
 >               are not searched for `#include <file>'.

OK, I'm beginning to see the light, please try the latest incarnation.
Thanks

Diego


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: faad_configure4.patch
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20031107/f1bfc347/attachment.txt>