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

[D Richard Felker III]

On Sat, Aug 30, 2003 at 10:10:59PM -0400, The Wanderer wrote:
> 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.

cvs update -d

BTW you should always use cvs update -dP. -d creates new directories
and -P prunes directories that no longer exist in cvs.

Rich