[MPlayer-dev-eng] Re: [Mplayer-cvslog] CVS: main/DOCS/en codecs.html,1.124,1.125

[The Wanderer]

Arpi wrote:
> Hi,
> 
> 
>>> IMO the argument for including faad is that the source package is
>>> usually broken and has idiotic dependencies (like libsoundfile
>>> or whatever). Same situation as dvdread/dvdcss when they were
>>> first imported to mplayer cvs. On the other hand, lame and vorbis
>>> libs are generally non-broken. Also lame is only needed for
>>> mencoder, which most people don't seem to use.
>>
>>I agree, provided we could make faad work without libsndfile.
> 
> 
> of course it works without it.
> i've just commited libfaad2 dir and the requires changes to 
> makefile/configure/ad_faad.c
> 
> please test the code, it should work on all platforms.

Don't know if I'm doing something wrong, but the compile now fails for
me in libmpcodecs/ad_faad.c; the errors seem to derive from not finding
"../libfaad2/faad.h". The libfaad2 directory does not exist on my
system; if something beyond a simple "cvs update" is required to create
it, I don't know what that is.

-- 
       The Wanderer

A government exists to serve its citizens, not to control them.