[FFmpeg-devel] [PATCH] Realmedia / RTSP (RDT)

[Aurelien Jacobs]

On Thu, 3 Jan 2008 00:00:28 +0100
Diego Biurrun <diego at biurrun.de> wrote:

> On Wed, Jan 02, 2008 at 11:50:50PM +0100, Luca Abeni wrote:
> >
> > --- libavformat/Makefile.orig	2008-01-02 23:29:02.000000000 +0100
> > +++ libavformat/Makefile	2008-01-02 23:41:51.000000000 +0100
> > @@ -122,8 +122,7 @@
> >  OBJS-$(CONFIG_ROQ_DEMUXER)               += idroq.o
> >  OBJS-$(CONFIG_ROQ_MUXER)                 += raw.o
> >  OBJS-$(CONFIG_RTP_MUXER)                 += rtp.o rtp_h264.o rtp_mpv.o rtp_aac.o
> > -OBJS-$(CONFIG_RTSP_DEMUXER)              += rtsp.o
> > -OBJS-$(CONFIG_SDP_DEMUXER)               += rtsp.o
> > +OBJS-$(CONFIG_SDP_DEMUXER)               += rtsp.o rtp.o rtp_h264.o rtp_mpv.o rtp_aac.o
> 
> Umm, yes, your patch makes the RTSP demuxer depend on the SDP demuxer,
> but still I think we should keep that line here.

I agree we should keep the RTSP line here.
Moreover, this still seems to be a dependency mess...
I feel there is a need for file splitting here (especially rtp.c).
Maybe I will try to have a deeper look at it.

Aurel