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 750721 - tools: gst-play: Print artist/album/title tags
tools: gst-play: Print artist/album/title tags
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-06-10 16:26 UTC by Vivia Nikolaidou
Modified: 2018-11-03 11:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch file (1.75 KB, patch)
2015-06-10 16:26 UTC, Vivia Nikolaidou
needs-work Details | Review

Description Vivia Nikolaidou 2015-06-10 16:26:24 UTC
Created attachment 304992 [details] [review]
Patch file

Print artist/album/title information for each track played.
Comment 1 Sebastian Dröge (slomo) 2015-09-23 08:18:15 UTC
Review of attachment 304992 [details] [review]:

Makes sense to me, why is this rejected? Should probably print all tags from the tag messages though, not just these few. See gst-launch code for some code that prints arbitrary tags

::: tools/gst-play.c
@@ +446,3 @@
+      if (!play->printed_tags) {
+        gchar *str;
+        if (gst_tag_list_get_string (tags, GST_TAG_ARTIST, &str)) {

You're leaking the strings here
Comment 2 Vivia Nikolaidou 2015-09-25 11:23:56 UTC
It basically needs to be redone while keeping a streaming radio in mind. New tags might fly by because the bitrate changed marginally, or because we've moved to the next track. We shouldn't reprint all the things if the bitrate changes marginally, but we should detect that we've moved to the next track.

Maybe remember title and artist and only print new information if these two change?

Also, just spoke with Tim, he said he's also happy with just artist/album/title and not all the things - personally I also slightly prefer printing just these, but don't really have a strong preference :)
Comment 3 Sebastian Dröge (slomo) 2015-09-25 11:41:34 UTC
Note that the version of gst-play in GstPlayer has this feature already
Comment 4 GStreamer system administrator 2018-11-03 11:38:06 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-base/issues/194.