Martin Michlmayr <[EMAIL PROTECTED]> writes:

> Out of interest, how does one use libmusicbrainz for cue sheets and
> does it get it right?  I looked at your flac-archive script, and while
> it uses some libmusicbrainz libraries, the cue sheet generation is
> quite similar to mkcue (and has the same bug).

Heh, I'm not surprised I made the same mistake.  I suppose you
were looking at fa-rip, but probably not the latest version.  See
svn://diplodocus.org/projects/audio/flac-archive/main .  I really
need to make a new release...

I suppose this patch should fix it?  Not tested, but I think it
matches your original patch.  I'm not sure why you removed the
first TRACK 01 AUDIO line, but neither am I sure why it's there.
I wrote that a long time ago, based on something I saw somewhere,
without actually understanding this stuff.

Thanks.

Index: fa-rip
===================================================================
--- fa-rip      (revision 2155)
+++ fa-rip      (working copy)
@@ -76,6 +76,9 @@
 
 from org.diplodocus.util import catch_EnvironmentError as c
 
+# http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=439790
+MSF_OFFSET = 150
+
 def mkcue(disc, trackcount=None):
     fp = c(file, 'cue', 'w')
     c(fp.write, 'FILE "dummy.wav" WAVE\n')
@@ -87,10 +90,9 @@
     else:
         trackcount = min(trackcount, disc.lastTrackNum)
 
-    pregap = disc.tracks[0][0]
     for i in xrange(disc.firstTrackNum, trackcount):
         offset = disc.tracks[i][0]
-        offset -= pregap
+        offset -= MSF_OFFSET
 
         minutes = seconds = 0
         sectors = offset % 75


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to