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 794779 - hlssink: Add a property to set start sequence number
hlssink: Add a property to set start sequence number
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.14.0
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-03-28 18:53 UTC by Yeongjin Jeong
Modified: 2018-11-03 14:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
hlssink: Add a property to set start sequence number (3.42 KB, patch)
2018-03-28 18:53 UTC, Yeongjin Jeong
none Details | Review
hlssink2: Add a property to set start sequence number (3.80 KB, text/plain)
2018-03-30 03:01 UTC, Daeseok Youn
  Details
hlssink2: Add a property to set start sequence number (3.79 KB, patch)
2018-03-30 05:29 UTC, Daeseok Youn
none Details | Review
hlssink: Add a property to set start sequence number (3.42 KB, patch)
2018-03-30 09:19 UTC, Yeongjin Jeong
none Details | Review

Description Yeongjin Jeong 2018-03-28 18:53:42 UTC
Created attachment 370259 [details] [review]
hlssink: Add a property to set start sequence number

This patch adds support to set start sequence number of first segment/file in playlist.

Before this patch, hlssink can't specifies starting sequence number of segment, it always had to be set to zero.
Comment 1 Sebastian Dröge (slomo) 2018-03-29 07:45:36 UTC
Can you add the same patch also for hlssink2?
Comment 2 Yeongjin Jeong 2018-03-29 12:45:45 UTC
(In reply to Sebastian Dröge (slomo) from comment #1)
> Can you add the same patch also for hlssink2?

The patch for hlssink2 will be uploaded soon by my co-worker 'Daeseok Youn'. We recently started the Gstreamer study together. :-) Is it okay?
Comment 3 Daeseok Youn 2018-03-30 02:36:15 UTC
Youngjin, 
There is a typo on patch set, see below:

+  g_object_class_install_property (gobject_class, PROP_START_NUMBER,
+      g_param_spec_uint ("start-number", "Start Numer",
                                                 ^^^^^

The "Numer" should be fixed to "Number".

Thanks.
Comment 4 Daeseok Youn 2018-03-30 03:01:04 UTC
Created attachment 370322 [details]
hlssink2: Add a property to set start sequence number

Introduce new property to set start sequence number for segment file on m3u8.

This patch provides same result with the patch(hlssink) created by Youngjin.

Thanks.
Comment 5 Daeseok Youn 2018-03-30 05:29:43 UTC
Created attachment 370323 [details] [review]
hlssink2: Add a property to set start sequence number

Update author's email address from previous one.
Comment 6 Yeongjin Jeong 2018-03-30 09:19:27 UTC
Created attachment 370334 [details] [review]
hlssink: Add a property to set start sequence number

Update typo on previous patch.

+  g_object_class_install_property (gobject_class, PROP_START_NUMBER,
-      g_param_spec_uint ("start-number", "Start Numer",
+      g_param_spec_uint ("start-number", "Start Number",
Comment 7 GStreamer system administrator 2018-11-03 14:19:57 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/674.