Bug#751800: mpg123: i486 decoder broken

[Simon Kirby]

On Mon, Jun 16, 2014 at 09:49:01PM +0200, Thomas Orgis wrote:

> The _dither part used to be for i586, but there is also a plain C
> variant in there now. The x86_dither is recommened for any
> i386-compatible CPU. It activates assembly optimizations for i586, MMX,
> SSE etc. at runtime. All i386-based decoders are merged in there ---
> except i486, which is indeed a special beast.

So, Debian maintainer: This would happen on my system with the diff below
applied. Not sure if i486 ever gets returned or if there are others to
consider, but mine is returning i386.

Simon-

--- mpg123-1.18.0/debian/rules.orig	2014-02-15 05:46:34.000000000 -0800
+++ mpg123-1.18.0/debian/rules	2014-06-24 17:01:47.865063715 -0700
@@ -21,6 +21,7 @@
 CONF_COMMON:=--enable-static=no --enable-shared --enable-lfs-alias \
 	--with-module-suffix=.so
 
+CONF_i386:=--with-cpu=x86_dither
 CONF_i486:=--with-cpu=x86_dither
 CONF_arm:=--with-cpu=generic_fpu