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

[Gildas Bazin]

On Friday 09 December 2005 00:13, Diego Biurrun wrote:
> On Thu, Dec 08, 2005 at 11:46:34PM +0000, Gildas Bazin wrote:
> > On Thursday 08 December 2005 00:42, Diego Biurrun wrote:
> > > 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.
> > > 
> > > 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.
> > 
> > Ok, here is a new patch which fixes --disable-muxers when used 
standalone (I 
> > had actually only tested it associated with --disable-protocols).
> > 
> > --disable-ffserver and --disable-protocols should also be fixed.
> 
> OK, we're getting closer, but any of
> 
> ./configure --enable-gpl --disable-demuxers
> ./configure --enable-gpl --disable-decoders
> ./configure --enable-gpl --disable-encoders
> 
> gives
> 
> gcc -Wl,--warn-common -rdynamic -g  -Wl,-E -o ffserver ffserver.o 
-L./libavformat -lavformat -L./libavcodec -lavcodec -L./libavutil -lavutil 
-lm -lz -ldl 
> ffserver.o: In function 
`http_parse_request':/home/diego/src/ffmpeg/ffserver.c:2361: undefined 
reference to `ffm_read_write_index'
> ffserver.o: In function 
`http_server':/home/diego/src/ffmpeg/ffserver.c:2424: undefined reference 
to `ffm_write_write_index'
> :/home/diego/src/ffmpeg/ffserver.c:2050: undefined reference to 
`ffm_set_write_index'
> ffserver.o: In function 
`build_feed_streams':/home/diego/src/ffmpeg/ffserver.c:3539: undefined 
reference to `ffm_read_write_index'
> collect2: ld returned 1 exit status
> make: *** [ffserver] Error 1
> 
> As I said, it might be easier to split this patch into independent parts
> that can be committed separately.  Then again, we should be pretty close
> now.
> 

Hi Diego,

I would be tempted to say that make distclean was missed between the 
compiles. I'm saying that because ffm_read_write_index is defined in 
libavformat/ffm.o and is only disabled when CONFIG_FFSERVER isn't defined.

I'll check this out this evening though.

--
Gildas