Re: [Gnash-dev] RTMP NetStream Patch

[Sandro Santilli]

Another thing: your patch lacks the oflaDemo.as file.

If it's easy for you, publishing your git branch will be helpful.
git request-pull --help # is your friend

--strk;


On Fri, Jun 21, 2013 at 03:15:57PM +0200, Sandro Santilli wrote:
> On Fri, Jun 21, 2013 at 04:50:11PM +0530, George Thomas wrote:
> > >
> > > Why is RTMP.cpp inolved in this at all, if libcurl would be enough
> > > to get the video our way ? Did you try using the command-line curl(1)
> > > to fetch the audio/video stream ?
> > 
> > RTMP looks to be supported with this patch.
> > 
> > Good news is I get sound and a streaming gnash with red5 now and I
> > was able to hear the audio.
> 
> Excellent! But, did does the RTMP stream also contain video ?
> 
> > The patch is attached with the additional change I did which made use of 
> > curl.
> > Shall I submit this patch in the patch submission page in savannah ?
> 
> Yes please. Make sure to target current master branch.
> 
> I noticed that your patch includes a currentFPS implementation,
> is that one related to RTMP support at all ? 
> 
> > Curl from command line works clean and downloads the flv.
> 
> Can the so-downloaded FLV be played with Gnash using a non-rtmp
> NetStream ? (ie: could be a lack of support for the specific FLV).
> NOTE: I'm assuming the FLV is supposed to also contain video here,
> let me know if I'm off-track.
> 
> --strk;

-- 

 http://www.cartodb.com - Map, analyze and build applications with your data

                                       ~~ http://strk.keybit.net