[gstreamer-bugs] [Bug 594035] Add HTTP Live Streaming playback support

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Nov 8 03:05:17 PST 2010


https://bugzilla.gnome.org/show_bug.cgi?id=594035
  GStreamer | don't know | git

--- Comment #20 from Andoni Morales <ylatuya at gmail.com> 2010-11-08 11:05:11 UTC ---
(In reply to comment #19)
> Review of attachment 174029 [details]:
> 
> Thank you very much Andoni for taking the time to improve greatly my initial
> work!
I must say that It was an excellent starting point :D
> 
> Just to highlight some of the changes that you made that might be
> controvorsial:
> 
> - make it a source, instead of a bin/"demuxer", implementing a "hls" URI
> handler
I though the element should be a source rather than a "demuxer" because it only
pushes fragments, like if it was reading from a file. I don't think gstreamer
supports playlists at the moment, and I haven't found any other element that
takes an uri, "demuxes" it and pushes the packets to a real demuxer. I don't
think it's very intuitive. And this could be implement with with a playlists
demuxer, whith a redirect.
 > - use Soup instead of gst_element_make_from_uri() for the "fetching" part.
The fetching part is very abstracted and could be easily replaced, but anyway I
think that's the right way of doing it, since it's pure HTTP (it's like making
souphttpsrc use inernally gst_element_make_from_uri() to support reading from
files) 
> - drop "appsrc" usage, replaced by a local fragment cache
It now uses an adapter and the element subclasses GstBaseSrc, which makes it
more or less the same. It could use a queue and push the whole fragment
downstream, but I don't if it's a good idea.
> - and many many things that I am sure improved a lot the element, and make it
> actually work :)
Yeiii!
> 
> I am glad you are pushing it forward, and I'll let someone more experienced
> with GStreamer comment the patch.
> Let's hope it's going to be merged soon! finger crossed!
Let see if it's approved and we have it in bad before the freeze :D

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.




More information about the Gstreamer-bugs mailing list