[MPlayer-users] mplayer fails to build with Live555 library

[Reimar Döffinger]

On Sat, Jul 27, 2013 at 05:16:09AM -0400, Josh wrote:
> On 7/27/2013 04:24, Carl Eugen Hoyos wrote:
> >Josh <joshf87 <at> live.com> writes:
> >
> >>On 7/25/2013 19:00, Umar Qureshey wrote:
> >>>I am trying to compile a recent svn version of
> >>>mplayer with the latest livemedia library
> >>>(http://live555.com/liveMedia/public/) so I can do
> >>>RTSP streaming.  However, the build fails:
> >>>
> >>MPlayer only supports the sync interface and it was
> >>dropped after Live555 4.30.2013. You'll need to use
> >>4.30.2013 or older.
> >
> >This is correct, before testing with unsupported
> >patches please try:
> >$ mplayer ffmpeg://rtsp://your.rtsp.stream/and/its/path
> >(with mplayer without live555 support)
> >
> >If this does not work, please report back, support is
> >much more likely than for the patch.
> >
> >Carl Eugen
> >
> >_______________________________________________
> >MPlayer-users mailing list
> >MPlayer-users at mplayerhq.hu
> >https://lists.mplayerhq.hu/mailman/listinfo/mplayer-users
> >
> >
> 
> I dropped Live555 from my builds for a bit, ffmpeg://rtsp:// worked
> just fine I didn't see any need for it but from a end-user
> standpoint they may get annoyed that they can't simply plug in a
> "rtsp://" url & may not go through the trouble of Googling and
> finding out they had to add "ffmpeg://". Might be time to update to
> Live555 async, or automatically have ffmpeg handle rtsp:// without
> "ffmpeg://".

Well, the live555 support and that the interaction is unclear is the
only reason it isn't like that.
Might make sense to do something like this if it works? Anyone care to test:
Index: stream/stream_ffmpeg.c
===================================================================
--- stream/stream_ffmpeg.c	(revision 36350)
+++ stream/stream_ffmpeg.c	(working copy)
@@ -154,7 +154,11 @@
   "",
   "",
   open_f,
-  { "ffmpeg", "rtmp", NULL },
+  { "ffmpeg", "rtmp",
+#ifndef CONFIG_LIVE555
+    "rtsp",
+#endif
+    NULL },
   NULL,
   1 // Urls are an option string
 };