Debian Bug report logs - #685679
mpg321: New buffered output feature makes mpg321 unusable

version graph

Package: mpg321; Maintainer for mpg321 is Nanakos Chrysostomos <nanakos@wired-net.gr>; Source for mpg321 is src:mpg321.

Reported by: Nanakos Chrysostomos <nanakos@wired-net.gr>

Date: Thu, 23 Aug 2012 11:00:01 UTC

Severity: important

Found in version mpg321/0.3.2-1.1

Reply or subscribe to this bug.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to debian-bugs-dist@lists.debian.org:
Bug#685679; Package mpg321. (Thu, 23 Aug 2012 11:00:04 GMT) Full text and rfc822 format available.

Acknowledgement sent to Nanakos Chrysostomos <nanakos@wired-net.gr>:
New Bug report received and forwarded. (Thu, 23 Aug 2012 11:00:04 GMT) Full text and rfc822 format available.

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

From: Nanakos Chrysostomos <nanakos@wired-net.gr>
To: Debian Bug Tracking System <submit@bugs.debian.org>
Subject: mpg321: New buffered output feature makes mpg321 unusable
Date: Thu, 23 Aug 2012 13:16:13 +0300
Package: mpg321
Version: 0.3.2-1.1
Severity: important

The new buffered output feature does not perform very well and its not very stable
making mpg321 unusable. It may also break other applications (frontend's) that use -b
option. The bug seems to be in semaphores syncing and locking.


-- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.0.0-2-amd64 (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages mpg321 depends on:
ii  libao4      1.1.0-2
ii  libasound2  1.0.25-4
ii  libc6       2.13-35
ii  libid3tag0  0.15.1b-10
ii  libmad0     0.15.1b-7
ii  zlib1g      1:1.2.7.dfsg-13

Versions of packages mpg321 recommends:
pn  libaudio-scrobbler-perl  <none>

mpg321 suggests no packages.

-- no debconf information



Information forwarded to debian-bugs-dist@lists.debian.org, Nanakos Chrysostomos <nanakos@wired-net.gr>:
Bug#685679; Package mpg321. (Mon, 17 Sep 2012 08:27:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to nanakos@wired-net.gr:
Extra info received and forwarded to list. Copy sent to Nanakos Chrysostomos <nanakos@wired-net.gr>. (Mon, 17 Sep 2012 08:27:05 GMT) Full text and rfc822 format available.

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

From: Nanakos Chrysostomos <nanakos@wired-net.gr>
To: <685679@bugs.debian.org>
Subject: Fwd: Re: mpg321 at mentors
Date: Mon, 17 Sep 2012 11:15:23 +0300
Dear Bart,

On Wed, 12 Sep 2012 19:08:18 +0000, Bart Martens <bartm@debian.org> wrote:
> Hi Nanakos,
> 
> I had a look at mpg321 at mentors uploaded there on 2012-08-27 09:13.
> 
> It is not clear to me on bug 685679 what you mean with "does not perform
> very
> well" and "not very stable".  I don't see how this makes "mpg321
unusable"
> and
> how it would "break other applications".  Please explain on the bug
report
> what's wrong and why you believe that disabling the -b option (actually
> making
> mpg321 silently ignore the -b option) would be the best approach in this
> context.
> 

I don't know if you have noticed an abnormal behavior when trying to use
the -b option but occasionally mpg321 crashes, stops playing the song,
plays the song
without syncing with the second process that outputs the song or the
reverse. Never starts
playing an mp3 file or crashes when someone tries to play a file that is
not an mp3. All those situations
does not happen when you are not using -b option. I am trying to solve the
problem but I haven't figured
where the fault is. So I prefer to silently as you say ignore the -b
option and preserve the package
in the previous state, so to provide a stable version that won't make
problems to users or other packages
that try or might use this new feature. 

> If the -b option does nothing then the manpage could be modified to
mention
> that.
> 

I will fix that and re-upload the package.

> The entries 0.3.2-2 and 0.3.2-1.1 in debian/changelog have lines with
one
> leading space too many.

Haven't noticed that, I will fix it.

> 
> The uncommenting of dh_testroot in debian/rules is not mentioned in
> debian/changelog.

I will fix that also.

Regards,
Chris.

> 
> Regards,
> 
> Bart Martens



Information forwarded to debian-bugs-dist@lists.debian.org, Nanakos Chrysostomos <nanakos@wired-net.gr>:
Bug#685679; Package mpg321. (Tue, 18 Sep 2012 05:09:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Bart Martens <bartm@debian.org>:
Extra info received and forwarded to list. Copy sent to Nanakos Chrysostomos <nanakos@wired-net.gr>. (Tue, 18 Sep 2012 05:09:03 GMT) Full text and rfc822 format available.

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

From: Bart Martens <bartm@debian.org>
To: Nanakos Chrysostomos <nanakos@wired-net.gr>, 685679@bugs.debian.org
Subject: mpg321: New buffered output feature makes mpg321 unusable
Date: Tue, 18 Sep 2012 05:06:38 +0000
Hello Nanakos,

I tried the -b option and I can confirm that it is easy to reproduce at least
one of the described problems with version 0.3.2-1.1.

   mpg321 -b 100 song.mp3

The program almost immediately ended with exit code 0 without playing the song.

In my opinion the severity of this bug is at least "serious".

I see that mpg321 already had the -b option in version 0.2.12-1, and I cannot
reproduce the problem with that version.

Disabling the -b option seems a good idea at first sight.  The program seems to
be usable without -b option.

I suggest to write a warning to standard error about the disabled -b option
when the -b option is tried, so that command line users immediately see that
the -b option is disabled.

I suggest to update the usage() function in mpg321.c to hide the -b option or
to mention that -b is disabled.

I suggest to alert the maintainers of packages in Debian using mpg321 so that
they can verify whether disabling the -b option can cause problems in their
packages.

Regards,

Bart Martens



Send a report that this bug log contains spam.


Debian bug tracking system administrator <owner@bugs.debian.org>. Last modified: Mon Apr 21 00:05:14 2014; Machine Name: buxtehude.debian.org

Debian Bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.