Rockbox

Tasklist

FS#2598 - APEv2 tags for MP3

Attached to Project: Rockbox
Opened by Frederic devernay (fredyd) - Tuesday, 26 July 2005, 14:12 GMT
Last edited by Jonathan Gordon (jdgordon) - Thursday, 17 April 2008, 13:06 GMT
Task Type Patches
Category ID3 / meta data
Status Closed
Assigned To No-one
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Hello,
APEv2 and id3v1 tags can be used to tag any file
format. When a file has both tags, id3v1 is at the end
and APEv2 just before it.
This patch fixes this and adds tags handling for any
file format (even wavpack, which had only APEv2 tags).

For mpa (mp1,mp2,mp3), if the user choses to use id3v1
tags in favor of id3v2, APEv2 tags won't be parsed either.
   apev2.patch (2 KiB)
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Thursday, 17 April 2008, 13:06 GMT
Reason for closing:  Rejected
Additional comments about closing:  Discussed many times and the official word is we dont want to support APEv2 (or any other non standard tags)
Comment by Frederic devernay (fredyd) - Thursday, 28 July 2005, 10:02 GMT

Since those parts of the code are moving fast, please ask me
for an up-to-date patch that applies to the current CVS :
fdy at melix dot net
Comment by Frederic devernay (fredyd) - Friday, 23 September 2005, 16:06 GMT

This new version is much smaller, and adds APEv2 tags
support to MP3 with very little modifications of the
existing code...
Comment by Loïc Lacombe (damaki) - Thursday, 09 February 2006, 22:00 GMT

I did a really small update so that it applies to current
CVS. I'm going to post the new version
Comment by Loïc Lacombe (damaki) - Thursday, 09 February 2006, 22:03 GMT

http://deathwind.free.fr/.fdgjkhkfgj/ApeV2_uptodate.patch
Comment by Nicolas Pennequin (nicolas_p) - Monday, 20 March 2006, 19:53 GMT
Here is an up-to-date version of the patch. damaki's seems to be reversed...
   APEv2.patch (2.3 KiB)
Comment by MichaelGiacomelli (saratoga) - Sunday, 09 April 2006, 03:44 GMT
Has this been merged into CVS ?
Comment by Ruben Petersen (Luckz) - Tuesday, 25 April 2006, 13:54 GMT
I wouldn't think so. As far as I know, as of mid-April, APE tags in MP3 files were not understood by any part of Rockbox.
Comment by Tan Yu Sheng (Farpenoodle) - Monday, 18 June 2007, 13:28 GMT
I hope I'm not the only one using this patch. Sync'd to new metadata file structure.
   APEv2.patch (3.4 KiB)
Comment by Craig Desrosiers (craig1) - Tuesday, 19 June 2007, 19:34 GMT
Thanks for updating this patch. I just started using it and it seems to work as advertised.
Comment by Tan Yu Sheng (Farpenoodle) - Saturday, 01 December 2007, 11:03 GMT
For those interested I updated the patch again. I also removed support for files other than MP3 that don't use APE tags since I don't use them. Should be trivial to add them back in if you need it though.
   APEv2.patch (2.6 KiB)
Comment by Craig Desrosiers (craig1) - Friday, 14 March 2008, 14:52 GMT
I updated the patch. Seems to be working for mp3 files.
   APEv2_2008-03-13.patch (2.8 KiB)
Comment by Dave Chapman (linuxstb) - Friday, 14 March 2008, 21:00 GMT
This feature/patch has been discussed amongst the developers a few times in IRC over the years, and the general view seems to be that there is no reason to support APEv2 tags in MP3 files - id3v2 are by far the most common tagging format used with MP3s, and (in line with Rockbox's approach for other file formats), we don't like to support multiple tagging formats in the same file (it adds complication - i.e. what tags to do you use if both are present, plus the requirement to perform extra seeking/reading when searching a file for tags).

So I think it's likely that this task will be rejected unless anyone can come up with a good justification for supporting both id3v2 and apev2 in MP3 files.
Comment by MichaelGiacomelli (saratoga) - Friday, 14 March 2008, 21:03 GMT
I think it should be left open as an invitation for someone to improve tag handling in rockbox such that more then one (or rather more then two if you count ID3v1 and V2) tag type could be parsed without adding extra seeks.
Comment by Dave Chapman (linuxstb) - Saturday, 15 March 2008, 01:44 GMT
@saratoga - the extra seek/read is only part of the issue, there is also the question of whether we want to support it at all. regardless of how it's implemented.

In any case, if an id3v2 tag is found, Rockbox won't look for an id3v1 tag anyway, meaning there is nothing to combine a seek for an APEv2 tag with. i.e. if Rockbox supported APEv2, it would always perform an extra seek/read for all files with an id3v2 tag.
Comment by Dave Chapman (linuxstb) - Thursday, 20 March 2008, 08:54 GMT
I'm underwhelmed by the support for this feature (i.e. there isn't any...). If no-one comments within the next few days, I'll reject it.
Comment by Craig Desrosiers (craig1) - Thursday, 20 March 2008, 19:31 GMT
I use MP3Gain on all of my mp3 files so that I get consistent playback levels on systems that don't support replaygain. The only problem is that MP3Gain writes its metadata in ApeV2 format. My workflow is significantly reduced with Rockbox supporting ApeV2 in mp3 files.
Comment by Dominik Riebeling (bluebrother) - Thursday, 20 March 2008, 19:39 GMT
mp3gain does _not_ save the replaygain information in an ape tag but only the undo data. See http://mp3gain.sourceforge.net/faq.php (as players not supporting replaygain benefit from the adjustment mp3gain did the adjustment simply can't be inside of any tag)
Comment by Tan Yu Sheng (Farpenoodle) - Friday, 21 March 2008, 09:28 GMT
No real reason other than that's what my files are tagged with being a longimte fb2k user. I figure if I'm going to tag my files in ID3v2 then I might as well make them compatible with everything else but that has problems so that's what I've stuck with rather than retag all my files.

Loading...