lame plugin should have (broken) tagging support removed

Bug #44213 reported by Andres Salomon
4
Affects Status Importance Assigned to Milestone
gst-plugins-ugly-multiverse0.10 (Ubuntu)
Fix Released
Medium
Sebastian Dröge

Bug Description

Binary package hint: gstreamer0.10-plugins-ugly-multiverse

The gstreamer lame plugin currently includes tagging support; however, it's currently broken. From what I've seen, if a tag has a length of more than 30 chars, it gets written as an id3v2 tag (and thus ends up in the mp3); if it has a shorter length, it's supposed to get written as an id3v1 tag (but it never gets written out due to bugs in the gstreamer plugin).

Upstream recommends using the id3mux plugin to write id3 tags; I've verified that this actually works. However, what that means is that if lame tagging works, the generated mp3 will end up w/ two sets of tags. Upstream has disabled tagging support in lame, in CVS; see http://bugzilla.gnome.org/show_bug.cgi?id=329184. It would be nice to have this removed from dapper, as well.

Revision history for this message
Andres Salomon (dilinger-deactivatedaccount) wrote :

Here's a patch, taken from CVS head and modified to apply to 0.10.3-1:

http://mouth.voxel.net/~dilinger/gstlame-against-0.10.3.patch

Revision history for this message
Sebastian Dröge (slomo) wrote :

thanks, I already had this patch locally and wanted to put it in at the weekend with my "backport millions of bugfixes from cvs" upload :)

Changed in gst-plugins-ugly-multiverse0.10:
assignee: nobody → slomo
status: Unconfirmed → In Progress
Revision history for this message
Sebastian Dröge (slomo) wrote :

Should be fixed with 0.10.3-2

Changed in gst-plugins-ugly-multiverse0.10:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.