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 753964 - jifmux: add "streamable" property
jifmux: add "streamable" property
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.4.5
Other Mac OS
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-08-22 14:25 UTC by minfrin
Modified: 2018-11-03 13:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-Add-support-for-the-streamable-property-which-allows.patch (4.01 KB, patch)
2015-08-22 14:25 UTC, minfrin
none Details | Review
Add support for the streamable property, which allows jifmux to support tags being written to a stream of JPEGs. By default only the first attempt to write a tag will appear in the JPEG. (4.39 KB, patch)
2015-11-13 18:03 UTC, minfrin
none Details | Review

Description minfrin 2015-08-22 14:25:08 UTC
Created attachment 309863 [details] [review]
0001-Add-support-for-the-streamable-property-which-allows.patch

By default jifmux uses the GST_TAG_MERGE_KEEP mode on incoming tag events, meaning that only the first attempt to set a tag counts.

This patch adds the streamable parameter, which if set to "true" will use GST_TAG_MERGE_REPLACE instead on tags.

This allows tags to be set on multiple JPEGs.

Patch is modelled on the flvmux element, which has identical behaviour.
Comment 1 Tim-Philipp Müller 2015-08-22 14:39:17 UTC
Comment on attachment 309863 [details] [review]
0001-Add-support-for-the-streamable-property-which-allows.patch

I don't know, this looks weird, and it's not what the 'streamable' property usually does or means.

It sounds like something isn't right with tag handling somewhere. That should just be fixed then IMHO. Also, why not set the merge mode on the element yourself from the application?
Comment 2 minfrin 2015-08-22 14:45:40 UTC
When you say "why not set the merge mode on the element yourself from the application" what application are you referring to?

The application I am using is gst-launch-1.0.

At the core of the problem inside the jifmux element is described by this function call, which is completely missing now and is added by the patch: http://gstreamer.freedesktop.org/data/doc/gstreamer/head/gstreamer/html/GstTagSetter.html#gst-tag-setter-set-tag-merge-mode

Right now, jifmux is only usable in one-shot applications like "create a thumbnail of this single video file".

For applications that involve many JPEGs, such as streams, jifmux is broken, as only the very first tag value set will appear in every JPEG produced.
Comment 3 minfrin 2015-11-13 18:03:50 UTC
Created attachment 315425 [details] [review]
Add support for the streamable property, which allows jifmux   to support tags being written to a stream of JPEGs. By  default only the  first attempt to write a tag will appear  in the JPEG.

The previous patch missed necessary changes to the header file.
Comment 4 GStreamer system administrator 2018-11-03 13:39:37 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/293.