[FFmpeg-devel] [PATCH] missing dependencies for rtsp and sdp demuxers?

[Luca Abeni]

Hi,

Jai Menon wrote:
> Hi,
> 
> With --disable-muxers, I get errors during linking which I think are
> because of missing dependencies. Attached patch fixes it for me, but
> I'm not sure if its entirely correct.
[...]
 > -OBJS-$(CONFIG_RTSP_DEMUXER)              += rdt.o rtsp.o
 > -OBJS-$(CONFIG_SDP_DEMUXER)               += rtsp.o rtp.o rtpdec.o rtp_h264.o rtp_vorbis.o
 > +OBJS-$(CONFIG_RTSP_DEMUXER)              += rdt.o rtsp.o rtp_vorbis.o rtp_asf.o
 > +OBJS-$(CONFIG_SDP_DEMUXER)               += rtsp.o rtp.o rtpdec.o rtp_h264.o rtp_vorbis.o rtp_asf.o
[...]

I am not sure about what happened to these dependencies, but this looks
wrong... IMHO, RTSP and SDP demuxers are built using exactly the same
.o files, so the two lines should be equal.
(I think the original plan was to have "OBJS-$(CONFIG_RTSP_DEMUXER)+=rtsp.o"
and have all the other rt(s)p related objects in CONFIG_SDP_DEMUXER,
because the RTSP demuxer depends on the SDP demuxer and it cannot be
enabled alone... But maybe RDT support broke this idea).



				Luca