Debian Bug report logs - #398805
libsdl-mixer1.2: fails with valid timidity.cfg config options

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Rémi Denis-Courmont <rdenis@simphalempin.com>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: libsdl-mixer1.2: fails with valid timidity.cfg config options

Date: Wed, 15 Nov 2006 19:18:17 +0200

Package: libsdl-mixer1.2
Version: 1.2.6-1.1+b2
Severity: important


	Hello,

libsdl-mixer1.2 tries to parse /etc/timidity/timidity.cfg which is part
of the timidity package. There is nothing wrong in that.

However, it fails to accept some directives that are valid for timidity
itself, such as the various "opt ..." directives that are commented out
in the current sample config file. This renders libsdl-mixer1.2
completely non-working with valid timidity.cfg.
This is vey inconvenient as timidity as currently in Debian needs these
directives otherwise it becomes EXTREMELY CPU-intensive.

libsdl-mixer1.2 should either use another config file or parse it
properly. Since it is my understanding that the timidity implementation
in libsdl-mixer does not implement all the fancy CPU-intensive features
of /usr/bin/timitity, it could simply ignore these options.


Regards,

-- System Information:
Debian Release: 4.0
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.18.1
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)

Versions of packages libsdl-mixer1.2 depends on:
ii  libc6              2.3.6.ds1-8           GNU C Library: Shared libraries
ii  libogg0            1.1.3-2               Ogg Bitstream Library
ii  libsdl1.2debian    1.2.11-7              Simple DirectMedia Layer
ii  libsmpeg0          0.4.5+cvs20030824-1.9 SDL MPEG Player Library - shared l
ii  libvorbis0a        1.1.2.dfsg-1.2        The Vorbis General Audio Compressi
ii  libvorbisfile3     1.1.2.dfsg-1.2        The Vorbis General Audio Compressi

libsdl-mixer1.2 recommends no packages.

-- no debconf information

Message #10 received at 398805@bugs.debian.org (full text, mbox, reply):

From: Brandon Barnes <winterknight@nerdshack.com>

To: 398805@bugs.debian.org

Subject: Here's a patch.

Date: Fri, 16 Feb 2007 12:11:24 -0800

tags 398805 +patch

Unfortunately, the original reporter did not give instructions on how
to reproduce this bug. I had a hard time figuring out when sdl-mixer
uses timidity.cfg. I never did figure that out.

However, I found the function that parses timidity.cfg. I incorporated
this function into a dummy program, that only parses the config file,
and nothing else. I managed to reproduce the bug in the dummy program,
and was able to come up with a solution that works there. However,
because I don't know how to reproduce the bug in an actual program, I
can't test if this patch has any side effects. I don't think it does;
it's pretty straight-forward.

Here it is:

diff -r sdl-mixer1.2-1.2.6.orig/timidity/timidity.c
sdl-mixer1.2-1.2.6/timidity/timidity.c 165,169c165
<     {
<      ctl->cmsg(CMSG_ERROR, VERB_NORMAL,
<         "%s: line %d: syntax error\n", name, line);
<      return -2;
<     }
---
>     continue; // Unrecognized option, skip this line

-Brandon

Message #15 received at 398805@bugs.debian.org (full text, mbox, reply):

From: Brandon Barnes <winterknight@nerdshack.com>

To: 398805@bugs.debian.org

Subject: Trying to mark this stupid thing as patched.

Date: Mon, 19 Feb 2007 19:20:01 -0800

tags 398805 +patch
thanks

Just trying to mark this stupid thing as patched. Pay no attention.

Message #22 received at 398805-close@bugs.debian.org (full text, mbox, reply):

From: Barry deFreese <bdefreese@debian.org>

To: 398805-close@bugs.debian.org

Subject: Bug#398805: fixed in sdl-mixer1.2 1.2.8-6

Date: Fri, 22 May 2009 15:17:28 +0000

Source: sdl-mixer1.2
Source-Version: 1.2.8-6

We believe that the bug you reported is fixed in the latest version of
sdl-mixer1.2, which is due to be installed in the Debian FTP archive:

libsdl-mixer1.2-dev_1.2.8-6_i386.deb
  to pool/main/s/sdl-mixer1.2/libsdl-mixer1.2-dev_1.2.8-6_i386.deb
libsdl-mixer1.2_1.2.8-6_i386.deb
  to pool/main/s/sdl-mixer1.2/libsdl-mixer1.2_1.2.8-6_i386.deb
sdl-mixer1.2_1.2.8-6.diff.gz
  to pool/main/s/sdl-mixer1.2/sdl-mixer1.2_1.2.8-6.diff.gz
sdl-mixer1.2_1.2.8-6.dsc
  to pool/main/s/sdl-mixer1.2/sdl-mixer1.2_1.2.8-6.dsc



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 398805@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Barry deFreese <bdefreese@debian.org> (supplier of updated sdl-mixer1.2 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 ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Fri, 22 May 2009 10:33:13 -0400
Source: sdl-mixer1.2
Binary: libsdl-mixer1.2 libsdl-mixer1.2-dev
Architecture: source i386
Version: 1.2.8-6
Distribution: unstable
Urgency: low
Maintainer: Debian SDL packages maintainers <pkg-sdl-maintainers@lists.alioth.debian.org>
Changed-By: Barry deFreese <bdefreese@debian.org>
Description: 
 libsdl-mixer1.2 - mixer library for Simple DirectMedia Layer 1.2
 libsdl-mixer1.2-dev - development files for SDL1.2 mixer library
Closes: 398805 453308
Changes: 
 sdl-mixer1.2 (1.2.8-6) unstable; urgency=low
 .
   [ Barry deFreese ]
   * Add myself to uploaders.
   * 04_timidity_cfg_opts.diff - (Closes: #398805).
     + Ignore unknown options in timidity.cfg.
     + Thanks to Brandon Barnes for the patch.
   * 05_fallback_to_freepats.diff - (Closes: #453308).
     + Use freepats config if timidity isn't present.
     + Thanks to Fabian Greffrath for the patch.
   * Bump debhelper build-dep and compat to 5.
   * Bump Standards Version to 3.8.1.
     + Move Homepage from package description to source stanza.
Checksums-Sha1: 
 77f92294217e7bf638340af14eb0da1477650ff8 1418 sdl-mixer1.2_1.2.8-6.dsc
 7311dc16aa65244090e3e92e5473bd19a7fd0c85 81219 sdl-mixer1.2_1.2.8-6.diff.gz
 1aaa66c42d4e6cdca2b1c38864931d44d067141a 93600 libsdl-mixer1.2_1.2.8-6_i386.deb
 4177f43a9f03593c8011aa76f3093a6e3bc84eaf 118242 libsdl-mixer1.2-dev_1.2.8-6_i386.deb
Checksums-Sha256: 
 e53c1bdd16374b5e79ecf1f1494effe78f92c891ff0290a1d7cf9d9ca3bcc080 1418 sdl-mixer1.2_1.2.8-6.dsc
 a60823a36395a5182dd6df6c201d9bfcb4534dfe3e2bd17d8467cabf5fe7bcbe 81219 sdl-mixer1.2_1.2.8-6.diff.gz
 0b7c5b64f0ce08f19a1508035d0e130523352e8c7f5a16054760c0c017c6e7d4 93600 libsdl-mixer1.2_1.2.8-6_i386.deb
 b9a95f84070e94ee2e24d2b08eab4fa0c151edf3295a0c35e03d8b7e0bde8285 118242 libsdl-mixer1.2-dev_1.2.8-6_i386.deb
Files: 
 3f41b4586df0e1317bb65db337f1bdd2 1418 libs optional sdl-mixer1.2_1.2.8-6.dsc
 ed315ef53d666d803ab2c947411170a2 81219 libs optional sdl-mixer1.2_1.2.8-6.diff.gz
 da9b4e9eca0cd1e09b6b4e7ad0e525d8 93600 libs optional libsdl-mixer1.2_1.2.8-6_i386.deb
 dfbf81d00938fd2fab1582d0df50ba8a 118242 libdevel optional libsdl-mixer1.2-dev_1.2.8-6_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkoWwVYACgkQ5ItltUs5T37BhgCfWBVReYC/9Fz26CdlDhiRdT/3
5TMAoNNNvODP9JV+GsHqpa02C3Glrh5j
=6cgB
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.