PREGAP=$(($(echo $OFFSETS | cut -f1 -d' ')))
TOTALTIME=$(( (($LEADOUT + $LEADIN + $PREGAP) / $CDFRAMES) - (($LEADIN + $PREGAP) / $CDFRAMES)))
+ vvecho "makeids: PREGAP $PREGAP, LEADIN $LEADIN, LEADOUT $LEADOUT"
+
# Calculate both the cddb discid *and* the musicbrainz discid
# now. We'll use the cddb discid for reference in most cases
# for consistency, but we also have the musicbrainz discid for
case $CDDBMETHOD in
*musicbrainz*)
# FIXME: don't assume the first track is 1
- MBDISCID=$($MUSICBRAINZ --command calcid --discinfo 1 $TRACKS $LEADIN $LEADOUT $OFFSETS)
+ MBDISCID=$($MUSICBRAINZ --command calcid --discinfo 1 $TRACKS $LEADIN $(($PREGAP + $LEADOUT)) $OFFSETS)
MBTRACKINFO="${MBDISCID} $((TRACKS)) ${COOKEDOFFSETS} $((($LEADOUT + $LEADIN + $IDMAGICNUM) / $CDFRAMES))"
;;
esac
abcde -o ogg -B
Or the appropriate settings in an ~/.abcde.conf file.
-
* Massive rework of CD lookup code so support multiple sources
better. Thanks to Gabriel Rosenkoetter for his initial idea in this
area, and to Tomasz GoliĆski on irc for initial inspiration on how
listed. All the results will be combined into one list at the end for
the user to select, just like would have previously worked for one
source only.
+ * Fix Musicbrainz ID calculation in makeids()
+ Apply fix suggested by petecollins24@gmail.com; add PREGAP to
+ LEADOUT to correct Musicbrainz ID calculation. Hopefully closes
+ Issue 54: https://abcde.einval.com/bugzilla/show_bug.cgi?id=54
abcde 2.8.1.