[Ffmpeg-devel] [PATCH] CONFIG_MUXERS

[Michael Niedermayer]

Hi

On Wed, Sep 21, 2005 at 11:04:23AM +0200, Diego Biurrun wrote:
> On Wed, Sep 21, 2005 at 03:00:22AM +0200, Diego Biurrun wrote:
> > On Tue, Sep 20, 2005 at 11:29:16AM +0200, Michael Niedermayer wrote:
> > > 
> > > On Tue, Sep 20, 2005 at 10:03:25AM +0200, Diego Biurrun wrote:
> > > > 
> > > > as recently discussed, this patch changes all occurrences of
> > > > CONFIG_ENCODERS in libavformat to CONFIG_MUXERS and adds support for
> > > > this to configure.  Individually disabling muxers would be nicer, but
> > > > this is a start (and I could hack it up quickly before going to bed ;).
> > > > 
> > > > OK to apply?
> > > 
> > > if it works (only muxers, only demuxers, both)
> > 
> > Well, it works no worse than before, but the build system is broken...
> > 
> > I'll fix tomorrow.
> 
> OK, here is a patch that fixes --disable-encoders and --disable-decoders
> in configure.  It compiles fine after my commit for the misplaced #ifdef
> yesterday, but it fails to link with errors like
> 
> gcc -Wl,--warn-common -rdynamic -g  -o ffmpeg_g ffmpeg.o cmdutils.o
> -L./libavformat -lavformat -L./libavcodec -lavcodec -L./libavutil
> -lavutil -lm -lz -ldl
> ./libavformat/libavformat.a(utils.o): In function `put_fifo':
> /home/diego/src/mplayer/ffmpeg/libavformat/utils.c:386: undefined
> reference to `put_buffer'
> 
> repeated multiple times.  This is not a regression however, since it
> obviously never worked before and will have to be fixed separately.
> 
> OK to apply the patch anyway or does this have to be fixed before?

no, CONFIG_EN/DECODERS should be set depending upon if there are any 
*CODERS enabled, not by --disable-en/decoders

[...]

-- 
Michael