[rtmpdump] [PATCH] Make rtmpdump handle full RTMP URLs with "-i"
hadess at hadess.net
Thu Jul 19 11:46:25 CEST 2012
On Thu, 2012-07-19 at 02:01 -0700, Howard Chu wrote:
> Bastien Nocera wrote:
> > On Wed, 2012-07-18 at 17:14 -0500, Steven Penny wrote:
> >> On Jul 18, 2012 Bastien Nocera wrote
> >>> Make rtmpdump handle full RTMP URLs with "-i" or --url. The library already
> >>> can handle those, so it makes debugging easier for users of the library.
> >> This is redundant code. RtmpDump already supports these options via
> >> --protocol
> >> --host
> >> --port
> >> --app
> >> --swfUrl
> >> --tcUrl
> >> Additionally, in many cases tcUrl is assumed based on the app/playpath. I say if
> >> you want to see this pass you will bring some more discussion to the table than
> >> "Short sentence commit message".
> > The library uses the parsed URL, but the command-line application can't?
> > Seems weird.
> Just a historical cruft. The command line tools were meant to be migrated to
> the new parser a long time ago.
Right, it was mainly to reply to Steve's opinion that those were already
> It would be nice to adapt rtmpgw too if you get a chance.
If that'll quicken pushing the patch upstream, certainly.
> > I'm writing code that generates those sort of URLs, which are understood
> > by applications like Totem, VLC, XBMC, or mplayer (through librtmp). It
> > makes complete sense that the low-level utility that's rtmpdump supports
> > it too.
More information about the rtmpdump