Package: eyed3
Version: 0.6.17-3
Severity: normal
Tags: patch

When run even without a request to be verbose, it emits a large amount of
text to stderr for every operation it performs.  Using stderr means that you
can't send it to /dev/null or you won't see actual errors.

Example output (nth run on the same file so it's no-op, but the first run
spammed as well):

{stdout}
02. Broderskapets Ring.mp3      [ 12.61 MB ]
-------------------------------------------------------------------------------
Time: 05:30     MPEG1, Layer III        [ 320 kb/s @ 44100 Hz - Joint stereo ]
-------------------------------------------------------------------------------
{stderr}
Removing ID3 v1.x tag: FAIL
Setting artist: Dimmu Borgir
Setting album: StormblÄst (re-recorded)
Setting title: Broderskapets Ring
Setting track: 2
Setting track genre: Black Metal
Setting year: 2003
No conversion necessary, tag is already version v2.4
Writing tag...
{stdout}
ID3 v2.4:
title: Broderskapets Ring               artist: Dimmu Borgir
album: StormblÄst (re-recorded)         year: 2003
track: 2                genre: Black Metal (id 138)


There's no "-q" option that would hush output that's not an (actual) warning
or an error.

Reading the program, I see that most of regular(!) output comes through a
function named printWarning().  In fact, not a single use of that function
produces an actual warning -- it's all regular confirmations of commands you
requested or at most reporting that an idempotent command was no-op.

Thus, I'd suggest this patch:

--- /usr/bin/eyeD3      2009-02-05 20:56:10.000000000 +0100
+++ eyeD3       2011-11-12 16:31:59.273160829 +0100
@@ -358,7 +358,7 @@
     sys.stdout.write(s + '\n');
 
 def printWarning(s):
-    sys.stderr.write(colors["warning"] + str(s) + colors["normal"] + '\n');
+    sys.stdout.write(colors["warning"] + str(s) + colors["normal"] + '\n');
 
 def printError(s):
     sys.stderr.write(colors["error"] + str(s) + colors["normal"] + '\n');


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

Kernel: Linux 3.0.0-2-amd64 (SMP w/6 CPU cores)
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 eyed3 depends on:
ii  python        2.7.2-9 
ii  python-eyed3  0.6.17-3

eyed3 recommends no packages.

eyed3 suggests no packages.

-- no debconf information



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to