Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 11194 - lame-3.93.0 sound quality problem (and fix)
Summary: lame-3.93.0 sound quality problem (and fix)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High critical
Assignee: phoen][x
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-11-25 07:10 UTC by Sebastian Dröge
Modified: 2002-12-12 12:04 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
the patch, to fix the problem (lame-3.93.0-npsytunefix.diff,1.07 KB, patch)
2002-11-25 07:11 UTC, Sebastian Dröge
Details | Diff
the ebuild, which uses the patch (lame-3.93.0-r1.ebuild,1.49 KB, text/plain)
2002-11-25 07:12 UTC, Sebastian Dröge
Details
patch to work around the -q0 problem (lame-3.93.0-q0fix.diff,864 bytes, patch)
2002-11-28 15:15 UTC, Sebastian Dröge
Details | Diff
ebuild with both patches (lame-3.93.0-r1.ebuild,1.56 KB, text/plain)
2002-11-28 15:16 UTC, Sebastian Dröge
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sebastian Dröge 2002-11-25 07:10:37 UTC
Hi,
look here for a description of the problem:
http://www.hydrogenaudio.org/index.php?act=ST&f=16&t=4455&s=92606aafd68b383e83a4e2c24d639774

Attached is a patch from the current lame CVS which solves this problem and an
ebuild which patches lame
lame will be soon released as version 3.93.1 with this patch

Bye
Comment 1 Sebastian Dröge 2002-11-25 07:11:26 UTC
Created attachment 5932 [details, diff]
the patch, to fix the problem
Comment 2 Sebastian Dröge 2002-11-25 07:12:03 UTC
Created attachment 5933 [details]
the ebuild, which uses the patch
Comment 3 Sebastian Dröge 2002-11-28 15:14:20 UTC
Now there's another patch in the lame cvs to work around a quality problem with -q0
The patch sets -q0 to -q1 
Comment 4 Sebastian Dröge 2002-11-28 15:15:30 UTC
Created attachment 6025 [details, diff]
patch to work around the -q0 problem
Comment 5 Sebastian Dröge 2002-11-28 15:16:35 UTC
Created attachment 6026 [details]
ebuild with both patches
Comment 6 phoen][x 2002-12-01 04:36:11 UTC
Hey Sebastian,

i added it to the official tree. Thanks for the contribution.

You might want to take a look at the src_unpack(), i made it more portage conform.

-phoen][x-
Comment 7 Jonathan Nall 2002-12-01 20:54:33 UTC
phoenix, 
this doesn't work for me:
bash-2.05b# ebuild lame-3.93.0-r1.ebuild unpack
>>> md5 ;-) lame-3.93.0.tar.gz
>>> Unpacking source...
>>> Unpacking lame-3.93.0.tar.gz
patching file libmp3lame/lame.c
Hunk #1 FAILED at 20.
1 out of 2 hunks FAILED -- saving rejects to file libmp3lame/lame.c.rej
patching file libmp3lame/psymodel.c
Hunk #1 FAILED at 19.
1 out of 3 hunks FAILED -- saving rejects to file libmp3lame/psymodel.c.rej
>>> Source unpacked.
bash-2.05b# 

Comment 8 phoen][x 2002-12-01 23:40:47 UTC
Impressive fix, no? ;)

Jonathan, give me an "emerge info" please.

-phoen][x-
Comment 9 Sebastian Dröge 2002-12-05 03:52:21 UTC
Hmm... it works for me with the standard lame-3.93.0.tar.bz2...
But now there's version 3.93.1 so this bug can be closed :)
Comment 10 Jonathan Nall 2002-12-06 17:50:38 UTC
phoen][x, just for you...

bash-2.05b# emerge info
Portage 2.0.44 (default-ppc-1.4, gcc-sh: line 1: /usr/bin/gcc: No such file or
directory, glibc-2.2.5-r7)
=================================================================
System uname: 2.4.19-r6 ppc 
USE="dvd xv jpeg mitshm gif ppc gnome-libs gdbm berkdb slang aalib guile pam
libwww ssl perl python esd imlib oggvorbis qt opengl mozilla X gtk gtk2 gnome
bonobo evo gb gtkhtml tcltk ncurses sdl png tiff gd mpeg quicktime oss xmms
flash encode crypt imap ldap mbox tcpd cups readline spell truetype xml xml2
pdflib plotutils tetex mysql samba gpm zlib pcmcia moznomail java directfb
-motif -nls -avi -alsa -arts -kde"
ARCH="ppc"
COMPILER="gcc3"
CHOST="powerpc-unknown-linux-gnu"
CFLAGS="-O2 -pipe -mcpu=7400 -maltivec -mabi=altivec -mpowerpc-gfxopt -fsigned-char"
CXXFLAGS="-O2 -pipe -mcpu=7400 -maltivec -mabi=altivec -mpowerpc-gfxopt
-fsigned-char"
ACCEPT_KEYWORDS="ppc ~ppc"
CONFIG_PROTECT="/etc /var/qmail/control /usr/share/config
/usr/kde/2/share/config /usr/kde/3/share/config
/usr/kde/3/share/config:/usr/share/config"
CONFIG_PROTECT_MASK="/etc/gconf /etc/env.d"
MAKEOPTS="-j2"
JDK_HOME="/opt/blackdown-jdk-1.3.1"
JAVA_HOME="/opt/blackdown-jdk-1.3.1"
AUTOCLEAN="yes"
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
GENTOO_MIRRORS="http://www.ibiblio.org/gentoo"
Comment 11 Jonathan Nall 2002-12-08 20:47:21 UTC
added myself to CC list
Comment 12 phoen][x 2002-12-11 10:22:53 UTC
Added lame-3.93.1 to the tree.
Give it a try, please.

-phoen][x-
Comment 13 Jonathan Nall 2002-12-12 06:23:11 UTC
phoenix,
3.93.1 compiles just fine. are you going to remove the patch from 3.93.0-r1 and
just let it have the quality issue?

Comment 14 phoen][x 2002-12-12 12:04:49 UTC
I removed lame-3.93.0-r1 and tagged lame-3.93.1 stable for x86.

I ~hope~ it's safe to close this bug now.

Just REOP it again if you need something. :)

-phoen][x-