Your message dated Sun, 21 Oct 2007 14:47:02 +0000
with message-id <[EMAIL PROTECTED]>
and subject line Bug#439790: fixed in mkcue 1-2.1
has caused the attached Bug report to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere. Please contact me immediately.)
Debian bug tracking system administrator
(administrator, Debian Bugs database)
--- Begin Message ---
Package: mkcue
Version: 1-2
Severity: important
As a comment in mkcue.cc says, mkcue assumes that the first song
always starts after 150 frames. However, this assumption is wrong.
As I understand it (from investigating this issue as it relates to
jack, see #319901), every CD has 150 frames that have to be ignored
(something known as the MSF offset). However, there is no guarantee
that a song starts exactly after 150 frames - it can start later.
When this happens, mkcue generates a bad cue file. As an effect of
this, a wrong FreeDB ID will also be generated. In my case this
means that the FreeDB ID fb108212 instead of e5108212 will be
computed.
You can see this pregap e.g. with cdparanoia. cdparanoia -Q shows:
track length begin copy pre ch
===========================================================
1. 13908 [03:05.33] 32 [00:00.32] no no 2
2. 20595 [04:34.45] 13940 [03:05.65] no no 2
i.e. cdinfo.FrameOffset[1] will have 182 (150 + 32).
The solution is to decrease cdinfo.FrameOffset[track] by 150 rather
than by cdinfo.FrameOffset[1]. With this change, my cue sheet looks
like this, which is better:
| TRACK 01 AUDIO
| INDEX 01 00:00:32
(see the patch below.)
There is one question though which I have not been able to answer
fully yet. How should this pregap of 32 frames be shown in the cue
sheet? It seems as if there were two options:
1) use INDEX 00, as in:
| TRACK 01 AUDIO
| INDEX 00 00:00:00
| INDEX 01 00:00:32
2) use PREGAP, as in:
| TRACK 01 AUDIO
| PREGAP 00:00:32
| INDEX 01 00:00:32
http://digitalx.org/cuesheetsyntax.php is not clear about which
version to favour and I don't know how I could ask.
--- mkcue-1.orig/mkcue.cc 2004-10-27 08:20:30.000000000 +0200
+++ mkcue-1/mkcue.cc 2007-08-27 13:38:19.000000000 +0200
@@ -54,16 +54,13 @@
}
puts("FILE \"dummy.wav\" WAVE");
- puts(" TRACK 01 AUDIO");
- puts(" INDEX 01 00:00:00");
#define min(x, y) ((x) < (y) ? (x) : (y))
- for (track = cdinfo.FirstTrack + 1;
+#define MSF_OFFSET 150
+ for (track = cdinfo.FirstTrack;
track <= min(trackcount, cdinfo.LastTrack);
track++) {
- /* There is frequently (always?) an offset of 150 sectors, so
- * subtract the first track's offset. */
- cdinfo.FrameOffset[track] -= cdinfo.FrameOffset[1];
+ cdinfo.FrameOffset[track] -= MSF_OFFSET;
minutes = seconds = sectors = 0;
sectors = cdinfo.FrameOffset[track] % 75;
-- System Information:
Debian Release: 4.0
APT prefers stable
APT policy: (500, 'stable')
Architecture: i386 (i686)
Shell: /bin/sh linked to /bin/bash
Kernel: Linux 2.6.18-4-686
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
--
Martin Michlmayr
http://www.cyrius.com/
--- End Message ---
--- Begin Message ---
Source: mkcue
Source-Version: 1-2.1
We believe that the bug you reported is fixed in the latest version of
mkcue, which is due to be installed in the Debian FTP archive:
mkcue_1-2.1.diff.gz
to pool/main/m/mkcue/mkcue_1-2.1.diff.gz
mkcue_1-2.1.dsc
to pool/main/m/mkcue/mkcue_1-2.1.dsc
mkcue_1-2.1_powerpc.deb
to pool/main/m/mkcue/mkcue_1-2.1_powerpc.deb
A summary of the changes between this version and the previous one is
attached.
Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to [EMAIL PROTECTED],
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Martin Michlmayr <[EMAIL PROTECTED]> (supplier of updated mkcue package)
(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [EMAIL PROTECTED])
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Format: 1.7
Date: Sun, 21 Oct 2007 16:12:13 +0200
Source: mkcue
Binary: mkcue
Architecture: source powerpc
Version: 1-2.1
Distribution: unstable
Urgency: low
Maintainer: Jesus Climent <[EMAIL PROTECTED]>
Changed-By: Martin Michlmayr <[EMAIL PROTECTED]>
Description:
mkcue - Generates a CUE sheet from a CD
Closes: 374823 439790
Changes:
mkcue (1-2.1) unstable; urgency=low
.
* NMU with the permission of the maintainer.
* Don't assume that the MSF offset is always 150 (Closes: #439790).
* Fix a typo in the man page, thanks A Costa (Closes: #374823).
Files:
3b92a3d5c8e21c37b312191bd5ec0af1 570 sound optional mkcue_1-2.1.dsc
341cedde19128847c95caa1b85fa3cd4 5102 sound optional mkcue_1-2.1.diff.gz
7bf77bc82f51a5f44dca9c92560d8279 9700 sound optional mkcue_1-2.1_powerpc.deb
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
iD8DBQFHG2RNKb5dImj9VJ8RAlkzAJwMpilh9WrocSPStwcy1op+A2D3ZQCfdNvO
fWQuasjT9uVKcUbmfgyYSts=
=JEsK
-----END PGP SIGNATURE-----
--- End Message ---
