Bug 15992 - ReplayGain info in APE tags not picked up for all mp3s
: ReplayGain info in APE tags not picked up for all mp3s
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Scanner
: 7.5.x
: Macintosh MacOS X 10.6
: P1 normal (vote)
: 7.5.x
Assigned To: Andy Grundman
: Audio::Scan
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-04-03 12:22 UTC by Richard van den Berg
Modified: 2010-06-18 10:12 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Richard van den Berg 2010-04-03 12:22:20 UTC
I tag all my mp3s on Linux using "mp3gain -c -s r". This stored the ReplayGain info in an APE tag without changing the mp3 data. Since SBS 7.4.0 the info is not picked up for all my files. This is repeatable: when I rescan my collection, the ReplayGain info is missing from the same files. Here are 2 example files:

http://no-spam-here.com/track1-not-ok.mp3
http://no-spam-here.com/track2-ok.mp3

My first guess was that this is a bug in MP3::Info but a simple $info=get_mp3tag($file,2,2,1); does pick up the info:

REPLAYGAIN_ALBUM_PEAK => 1.135093
TCOM => 
TOPE => 
TYER => 2010
REPLAYGAIN_ALBUM_GAIN => -8.350000 dB
REPLAYGAIN_TRACK_PEAK => 1.077664
GENRE => 
TPE1 => Balthazar
TAGVERSION => ID3v2.3.0
TCOP => 
MP3GAIN_MINMAX => 145,255
TALB => Applause 
TRCK => 01
MP3GAIN_ALBUM_MINMAX => 141,255
TIT2 => Fifteen Floors
TCON => Alt. Rock
WXXX => 
REPLAYGAIN_TRACK_GAIN => -8.580000 dB
COMM => . . .
TENC =>
Comment 1 Andy Grundman 2010-04-03 13:03:37 UTC
I'll look at this. Btw MP3::Info is no longer used.

It would sure be nice if mp3gain wrote proper ID3 tags instead of APE tags...
Comment 2 Andy Grundman 2010-04-03 13:44:00 UTC
OK the problem is there is a Lyricsv2 tag in there between the ID3v1 and APEv2 tags, and that's not picked up at the moment.  Will fix for 7.5.1.
Comment 3 SVN Bot 2010-04-13 05:54:13 UTC
 == Auto-comment from SVN commit #655 to the opensource repo by agrundman ==
 == http://svn.slimdevices.com/opensource?view=revision&revision=655 ==

Fixed bug 15992, read APE tags correctly when a Lyricsv2 tag is present
Comment 4 Richard van den Berg 2010-05-13 09:36:09 UTC
I am not running SBS 7.5.1 - r30670 on OS X which has this fix. It works great for the most part, but there are still tracks where the ReplayGain info is not read. One example is: http://no-spam-here.com/track3-not-ok.mp3

If I need a to submit new bug report instead, let me know.
Comment 5 Andy Grundman 2010-05-13 10:14:45 UTC
Tested that file on 7.5.1, it scans perfectly for me.
Comment 6 Richard van den Berg 2010-05-13 11:14:37 UTC
I upgraded to 7.5.1 - r30739 and rescanned my collection, and it is fine now. Strange. I used MySQL Workbench to check my full collection in SBS database, and all ReplayGain info is there now. Thanks!
Comment 7 Richard van den Berg 2010-06-03 12:47:05 UTC
Sorry to reopen this once again. This track http://no-spam-here.com/track4-not-ok.mp3 only shows a -9.24 dB adjustment for Albums in SBS. The -6 dB adjustment for the track is not shown, and not used when playing random tracks. I tested this with SBS 7.5.1 r30822 on OS X and Linux.
Comment 8 Richard van den Berg 2010-06-03 13:13:25 UTC
Here is another one, not showing any ReplayGain info in SBS: http://no-spam-here.com/track5-not-ok.mp3
Comment 9 SVN Bot 2010-06-07 11:24:26 UTC
 == Auto-comment from SVN commit #700 to the opensource repo by agrundman ==
 == http://svn.slimdevices.com/opensource?view=revision&revision=700 ==

Fixed bug 15992, APE tag could be ignored if ID3v1 was present in some cases
Comment 10 Richard van den Berg 2010-06-18 10:12:06 UTC
Thanks again. I'd like to test this fix, but 7.5.2-30889 and 7.6.0-30918 downloaded on June 18th still have Audio::Scan 0.82. The fix should be in version 0.83 according to the Changes file.