[Ffmpeg-devel] [PATCH] small build cleanup + new configure option

[Diego Biurrun]

On Sat, Dec 03, 2005 at 01:09:59AM +0000, Gildas Bazin wrote:
> 
> Here is a small patch that makes sure --disable-muxers  and 
> --disable-ffserver work properly.

So --disable-muxers works for you?  I get tons of errors with and without
your patch.

--disable-ffserver works without your patch, with your patch I get

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
ffmpeg.o: In function `opt_default':/home/diego/src/ffmpeg/ffmpeg.c:3974: undefined reference to `ffm_nopts'
:/home/diego/src/ffmpeg/ffmpeg.c:3974: undefined reference to `ffm_nopts'
ffmpeg.o: In function `opt_target':/home/diego/src/ffmpeg/ffmpeg.c:3974: undefined reference to `ffm_nopts'
:/home/diego/src/ffmpeg/ffmpeg.c:3974: undefined reference to `ffm_nopts'
collect2: ld returned 1 exit status
make: *** [ffmpeg_g] Error 1

> A new --disable-protocols option is also added to the build to disable the 
> I/O protocols support from libavformat. For instance VLC media player can 
> make use of the libavformat demuxers but uses its own I/O routines, 
> furthermore the libavformat I/O protocols routines don't compile under 
> WinCE so this option can be handy.

./configure --disable-protocols; make

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 `av_open_input_file':/home/diego/src/ffmpeg/libavformat/utils.c:590: undefined reference to `redir_demux'
:/home/diego/src/ffmpeg/libavformat/utils.c:590: undefined reference to `redir_demux'
:/home/diego/src/ffmpeg/libavformat/utils.c:591: undefined reference to `redir_open'
collect2: ld returned 1 exit status
make: *** [ffmpeg_g] Error 1

I like the idea of your patch and would very much like to apply it, but
there still seem to be some bugs to work out.  Perhaps you can separate
the --disable-protocols part to make this easier.

Diego