Package: ripit Version: 3.9.0-2 Severity: normal When querying for track info using MusicBrainz, ripit uses the music brainz id instead of the discid, which leads to the return of tracks for the first disc instead of tracks for the second disk. The trivial patch attached searches for track information using the discid instead of the mbid, solving this issue.
Don Armstrong -- [A] theory is falsifiable [(and therefore scientific) only] if the class of its potential falsifiers is not empty. -- Sir Karl Popper _The Logic of Scientific Discovery_ ยง21 http://www.donarmstrong.com http://rzlab.ucr.edu
--- /home/don/bin/ripit 2011-08-26 10:01:58.000000000 -0700 +++ /usr/bin/ripit 2011-04-26 05:26:31.000000000 -0700 @@ -1532,11 +1531,11 @@ print TMP "barcode: $barcode\n" if($barcode); # Retrieve the track list. - eval {$mbid_response = $service->search({ DISCID => $discid, INC => 'tracks' });}; + eval {$mbid_response = $service->search({ MBID => $mbid, INC => 'tracks' });}; if($@){ print "\nMusicBrainz lookup failed... 2nd try in 3s:\n" if($verbose > 3); sleep 3; - eval {$mbid_response = $service->search({ DISCID => $discid, INC => 'tracks' });}; + eval {$mbid_response = $service->search({ MBID => $mbid, INC => 'tracks' });}; if($@){ print "\nMusicBrainz lookup failed! Using freedb instead.\n" if($verbose > 3);