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 710203 - hlssink doesnt fragment audio-only streams
hlssink doesnt fragment audio-only streams
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-10-15 17:06 UTC by Javier Jardón (IRC: jjardon)
Modified: 2018-11-03 13:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
mpegtsmux: respect force-key-unit when dealing with audio only (2.47 KB, patch)
2015-02-05 16:09 UTC, Thiago Sousa Santos
none Details | Review

Description Javier Jardón (IRC: jjardon) 2013-10-15 17:06:05 UTC
Test case:

gst-launch-1.0 audiotestsrc ! faac ! mpegtsmux ! hlssink target-duration=2

doesnt generate new segments, while

gst-launch-1.0 videotestsrc ! x264enc ! mpegtsmux ! hlssink target-duration=2

works without problems
Comment 1 Andoni Morales 2013-10-16 14:00:24 UTC
faac is not forwarding GstForceKeyUnits downstream and mpegtsmux is not not tracking upstream event in the src pad, that's why it doesn't work.

This patch for qtmux does that correctly if you want to take a look:
https://bug660260.bugzilla-attachments.gnome.org/attachment.cgi?id=252750

I think for audio only streams you should be using AAC + ID3 according to the spec, which won't work either, in this case because id3mux doesn't support GstForceKeyUnit events.
Comment 2 Javier Jardón (IRC: jjardon) 2013-10-16 16:15:29 UTC
Thanks Andoni,

About mpegtsmux, seems that there is already a fix in bug #696032 ?
Comment 3 Thiago Sousa Santos 2015-02-05 16:09:55 UTC
Created attachment 296203 [details] [review]
mpegtsmux: respect force-key-unit when dealing with audio only

This makes it work with the example from the reporter but even some audio
formats require headers so there is some room for a 'key-unit' event for
audio as well. The problem is that the event is defined in the video library.

So should we create an audio or just go with the video one for audio?
Comment 4 Tim-Philipp Müller 2015-11-16 09:52:32 UTC
This whole reliance on the force-key-unit events to do the actual switch-over seems broken to me. I don't think these make sense for audio-only, and I don't think it makes sense for audio encoders to handle them or create audio-specific ones.
Comment 5 GStreamer system administrator 2018-11-03 13:18:26 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/113.