After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 672261 - [PLUGIN-MOVE] Move hlsdemux/dashdemux/mssdemux to gst-plugins-good
[PLUGIN-MOVE] Move hlsdemux/dashdemux/mssdemux to gst-plugins-good
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-03-16 20:35 UTC by Thibault Saunier
Modified: 2018-11-03 13:11 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Thibault Saunier 2012-03-16 20:35:03 UTC
We should probably move hlsdemux to -good at some time.

On IRC it has been said that we should wait for #657790 to be closed, and we might want to rename it from hls to something more generic (fragmented is the name of the plugin itself, just the folder is called hls). Ideally adding another fragmented streaming protocol with the help of the new base class.

Also there is no test suite yet but there is some work going on there.
Comment 1 Wim Taymans 2013-03-14 12:53:01 UTC
It needs a lot of cleanups before I would move this plugin..

1) the downloading of complete fragments is not ideal. You need to handle
   buffering in smaller chuncks to give good feedback about the buffering.

2) it never rebuffers when the queues run empty

3) it stops threads when PAUSED (now fixed in git)

4) seeking code is broken. It should use GstSegment to track the playback
   position.

5) the thread handling looks weird and buggy, it sets variables outside of
   locks and it looks like it can be a lot simpler.
Comment 2 Olivier Crête 2013-09-17 23:34:12 UTC
We also need some test cases, seeking was regressed (completely broken) for quite a while.
Comment 3 Sebastian Dröge (slomo) 2016-11-16 13:25:55 UTC
Moving it might be a good idea now, it's widely used. But I wouldn't want to expose the library API (adaptivedemux), that should be internal (so probably should move all 3 plugins into one with that).
Comment 4 Tim-Philipp Müller 2016-11-16 16:45:02 UTC
Maybe even -base, as it (a) has no adaptive streaming stuff yet, (b) the libs thing fits better with -base even if not exposed, (c) is pretty important.
Comment 5 GStreamer system administrator 2018-11-03 13:11:16 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/64.