[FFmpeg-devel] [PATCH] fix sdp(file) vs rtsp dep

[Luca Abeni]

Diego Biurrun wrote:
> On Mon, Oct 19, 2009 at 09:09:36PM +0200, Luca Abeni wrote:
>> [...]
>>
>> I finally found some time to run some tests, and it seems to me that the 
>> situation is much simpler than expected: I might be wrong, but I think 
>> the SDP demuxer is just missing a dependency on rdt.o in 
>> libavformat/Makefile.
> 
> While this is correct, you are missing the point.
> 
> configure:
> 
> rtsp_demuxer_deps="sdp_demuxer"
> 
> 
> libavformat/rtsp.c:
> 
> static int sdp_read_packet(AVFormatContext *s,
>                             AVPacket *pkt)
> {
>     return rtsp_read_packet(s, pkt);
> }
> 
> 
> So the SDP demuxer depends on RTSP functions, while configure claims
> that the RTSP demuxer depends on the SDP demuxer.  This is a circle.

Yes, you are right: the functions' naming is wrong. I do not know why
they are named like this (nor why this kind of wrapper is used...).
Thanks for pointing this out.

Anyway, I think this is easily fixable; I'll send a patch later today
or tomorrow, when I find some time for testing it.


				Luca