[FFmpeg-cvslog] r25343 - trunk/libavformat/rtsp.c

[Martin Storsjö]

On Tue, 5 Oct 2010, Diego Biurrun wrote:

> On Tue, Oct 05, 2010 at 10:48:30PM +0300, Martin Storsj? wrote:
> > On Tue, 5 Oct 2010, Martin Storsj? wrote:
> > > Btw, any objections to changing the rtsp/sdp _deps to _select? Then one 
> > > would be able to simply do --disable-everything --enable-demuxer=rtsp and 
> > > get the right things enabled, without having to hunt for the deps 
> > > manually.
> > 
> > This question still is open. :-)
> 
> Go for it.

Done.

Now most of the --disable-everything + enable only one demuxer/muxer works 
as you'd expect.

There's still one weird case, though. If I do --disable-everything 
--enable-muxer=rtsp, I get this build error:

Undefined symbols:
  "_ff_get_wav_header", referenced from:
      _asf_read_header in libavformat.a(asfdec.o)

The reason for this is that the rtsp muxer via it's shared common code 
with the rtsp demuxer depends on a bunch of demuxers, too. The asf demuxer 
uses code from riff.c, and the ff_get_wav_header function in riff.c is 
within CONFIG_DEMUXERS. And in this config setup, CONFIG_DEMUXERS actually 
is 0, even if there are demuxers enabled, since no demuxers were 
explicitly enabled, only implicitly via dependencies.

I remember looking into this issue some time ago, but wasn't able to solve 
it in any clean way.

// Martin