Musicbrainz lookup not working if data track present

Bug #989741 reported by Eike Thaden
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
soundkonverter
Fix Released
Undecided
Unassigned

Bug Description

The lookup of metadata based on the discid via Musicbrainz is not working on Ubuntu 12.04 (AMD64) for audio discs with data tracks.

Revision history for this message
Eike Thaden (ethaden) wrote :

I think that the code used to prepare the input for the libkcddb discid calculation is flawed at least for Musicbrainz lookups. The patch adds detection of data tracks which are then ignored when preparing the list of track offsets. The behavior for audio discs without data tracks remains unchanged. I currently don't know whether or not the modified discid calculation for discs with data tracks will work for freedb lookups, too. Strange: The discid calculation algorithm presented on the Musicbrainz web seite (http://musicbrainz.org/doc/Disc_ID_Calculation) did not yield any results for my test audio disc when using their proposed method for calculation of the lead out track offset. Instead, the lookup was successful if all data tracks are ignored and the track offset calculation is left unchanged (using method "cdda_track_firstsector").

Revision history for this message
Eike Thaden (ethaden) wrote :

Typo: The used method is called "cdda_disc_lastsector".

Revision history for this message
Daniel Faust (hessijames) wrote :

Thank you very much.
I never tested it with a cd containing data tracks.

The lead-out track issue is indeed strange. I could imagine that the application which submitted the data was broken. But testing that would be virtually impossible for me so I'm happy that it works now at least in your case.

Revision history for this message
Eike Thaden (ethaden) wrote : Re: [Bug 989741] Re: Musicbrainz lookup not working if data track present

Am 27.04.2012 20:19, schrieb HessiJames:
> Thank you very much.
> I never tested it with a cd containing data tracks.
>
> The lead-out track issue is indeed strange. I could imagine that the
> application which submitted the data was broken. But testing that would
> be virtually impossible for me so I'm happy that it works now at least
> in your case.
>
It seems that the patch does not work for the native Musicbrainz
lookups, but only for the lookups done via the Musicbrainz freedb
gateway (using the freedb protocol).

Changed in soundkonverter:
status: New → Fix Committed
Changed in soundkonverter:
status: Fix Committed → 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.