Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 631852 - app-i18n/uchardet-0.0.6: test failure: Test #44: th:tis-620 failed
Summary: app-i18n/uchardet-0.0.6: test failure: Test #44: th:tis-620 failed
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: Normal normal (vote)
Assignee: Coacher
URL: https://bugs.freedesktop.org/show_bug...
Whiteboard:
Keywords: TESTFAILURE, UPSTREAM
Depends on:
Blocks:
 
Reported: 2017-09-23 16:19 UTC by Thomas Deutschmann (RETIRED)
Modified: 2017-12-19 13:21 UTC (History)
2 users (show)

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


Attachments
build.log (build.log,35.21 KB, text/plain)
2017-09-23 16:19 UTC, Thomas Deutschmann (RETIRED)
Details
LastTest.log (LastTest.log,35.92 KB, text/plain)
2017-09-23 16:20 UTC, Thomas Deutschmann (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Deutschmann (RETIRED) gentoo-dev 2017-09-23 16:19:33 UTC
Created attachment 496174 [details]
build.log

44/54 Testing: th:tis-620
44/54 Test: th:tis-620
Command: "/var/tmp/portage/app-i18n/uchardet-0.0.6/work/uchardet-0.0.6_build/test/uchardet-tests" "/var/tmp/portage/app-i18n/uchardet-0.0.6/work/uchardet-0.0.6/test/th/tis-620.txt"
Directory: /var/tmp/portage/app-i18n/uchardet-0.0.6/work/uchardet-0.0.6_build/test
"th:tis-620" start time: Sep 23 18:14 CEST
Output:
----------------------------------------------------------
<end of output>
Test time =   0.01 sec
----------------------------------------------------------
Test Failed.
"th:tis-620" end time: Sep 23 18:14 CEST
"th:tis-620" time elapsed: 00:00:00
---------------------------------------
Comment 1 Thomas Deutschmann (RETIRED) gentoo-dev 2017-09-23 16:20:22 UTC
Created attachment 496176 [details]
LastTest.log

Portage 2.3.8 (python 3.4.5-final-0, default/linux/x86/13.0, gcc-5.4.0, glibc-2.23-r4, 4.12.12-gentoo i686)
=================================================================
System uname: Linux-4.12.12-gentoo-i686-Intel-R-_Core-TM-_i7-3770K_CPU_@_3.50GHz-with-gentoo-2.3
KiB Mem:     3108120 total,   2337104 free
KiB Swap:     488276 total,    488276 free
Timestamp of repository gentoo: Sat, 23 Sep 2017 15:47:43 +0000
Head commit of repository gentoo: 751d60d0f1fdee1f8c56f1c4dbfc324022e7ddbd

sh bash 4.3_p48-r1
ld GNU ld (Gentoo 2.28.1 p1.0) 2.28.1
app-shells/bash:          4.3_p48-r1::gentoo
dev-java/java-config:     2.2.0-r3::gentoo
dev-lang/perl:            5.24.1-r2::gentoo
dev-lang/python:          2.7.12::gentoo, 3.4.5::gentoo
dev-util/cmake:           3.7.2::gentoo
dev-util/pkgconfig:       0.28-r2::gentoo
sys-apps/baselayout:      2.3::gentoo
sys-apps/openrc:          0.28::gentoo
sys-apps/sandbox:         2.10-r3::gentoo
sys-devel/autoconf:       2.13::gentoo, 2.69::gentoo
sys-devel/automake:       1.15-r2::gentoo
sys-devel/binutils:       2.28-r2::gentoo, 2.28.1::gentoo
sys-devel/gcc:            5.4.0-r3::gentoo
sys-devel/gcc-config:     1.7.3::gentoo
sys-devel/libtool:        2.4.6-r3::gentoo
sys-devel/make:           4.2.1::gentoo
sys-kernel/linux-headers: 4.4::gentoo (virtual/os-headers)
sys-libs/glibc:           2.23-r4::gentoo
Repositories:

gentoo
    location: /usr/portage
    sync-type: git
    sync-uri: https://github.com/gentoo-mirror/gentoo.git
    priority: -1000

ABI="x86"
ABI_X86="32"
ACCEPT_KEYWORDS="x86"
ACCEPT_LICENSE="* -@EULA"
ACCEPT_PROPERTIES="*"
ACCEPT_RESTRICT="*"
ANT_HOME="/usr/share/ant"
ARCH="x86"
CBUILD="i686-pc-linux-gnu"
CFLAGS="-O2 -pipe -march=ivybridge -mtune=ivybridge -mno-xsaveopt"
CHOST="i686-pc-linux-gnu"
CHOST_x86="i686-pc-linux-gnu"
COLLISION_IGNORE="/lib/modules/* *.py[co] *$py.class */dropin.cache"
CONFIG_PROTECT="/etc /usr/share/gnupg/qualified.txt /var/bind"
CPU_FLAGS_X86="aes avx mmx mmxext popcnt sse sse2 sse3 sse4_1 sse4_2 ssse3"
CXXFLAGS="-O2 -pipe -march=ivybridge -mtune=ivybridge -mno-xsaveopt"
DEFAULT_ABI="x86"
EDITOR="/usr/bin/mcedit"
ELIBC="glibc"
EPREFIX=""
EROOT="/"
FACTERDIR="/usr/lib"
FCFLAGS="-O2 -march=i686 -pipe"
FEATURES="assume-digests binpkg-logs cgroup config-protect-if-modified distlocks downgrade-backup ebuild-locks fixlafiles merge-sync multilib-strict news parallel-fetch preserve-libs protect-owned sandbox sfperms strict unknown-features-warn unmerge-logs unmerge-orphans userfetch userpriv usersandbox usersync xattr"
FFLAGS="-O2 -march=i686 -pipe"
GCC_SPECS=""
GRUB_PLATFORMS="efi-32 pc"
GSETTINGS_BACKEND="dconf"
HOME="/root"
INFOPATH="/usr/share/info:/usr/share/gcc-data/i686-pc-linux-gnu/5.4.0/info:/usr/share/binutils-data/i686-pc-linux-gnu/2.28.1/info"
INPUT_DEVICES="libinput keyboard mouse"
IUSE_IMPLICIT="abi_x86_32 prefix prefix-chain prefix-guest"
JAVAC="/etc/java-config-2/current-system-vm/bin/javac"
JAVA_HOME="/etc/java-config-2/current-system-vm"
JDK_HOME="/etc/java-config-2/current-system-vm"
KERNEL="linux"
L10N="en en-US de de-DE"
LANG="en_US.UTF-8"
LC_ALL="en_US.UTF-8"
LC_MESSAGES="C"
LC_PAPER="de_DE.UTF-8"
LDFLAGS="-Wl,-O1 -Wl,--as-needed"
LIBDIR_x86="lib"
LINGUAS="en de"
LOGNAME="root"
MAIL="/var/mail/root"
MAKEOPTS="--jobs 5 --load-average 7.95"
MULTILIB_ABIS="x86"
MULTIOSDIRS="../lib"
NOCOLOR="true"
OFFICE_IMPLEMENTATION="libreoffice"
OLDPWD="/root"
OPENGL_PROFILE="xorg-x11"
PAGER="/usr/bin/less"
PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/opt/bin:/usr/i686-pc-linux-gnu/gcc-bin/5.4.0"
PHP_TARGETS="php5-6"
POSTGRES_TARGETS="postgres9_5"
PWD="/usr/portage/media-video"
PYTHONDONTWRITEBYTECODE="1"
PYTHON_SINGLE_TARGET="python3_4"
PYTHON_TARGETS="python2_7 python3_4"
ROOT="/"
ROOTPATH="/opt/bin:/usr/i686-pc-linux-gnu/gcc-bin/5.4.0"
RUBY_TARGETS="ruby22"
SHELL="/bin/bash"
SHLVL="2"
SSH_TTY="/dev/pts/0"
TERM="screen"
TMUX="/tmp/tmux-0/default,4589,0"
TMUX_PANE="%1"
TWISTED_DISABLE_WRITING_OF_PLUGIN_CACHE="1"
USER="root"
USERLAND="GNU"
VIDEO_CARDS="intel"
Comment 2 Coacher 2017-10-08 20:19:22 UTC
PR: https://github.com/gentoo/gentoo/pull/5890

We disable the broken test for now.
Whenever upstream delivers a proper fix, we'll backport it.
Comment 3 Larry the Git Cow gentoo-dev 2017-10-08 22:02:06 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ace0b97f971549f85648facf01ef4542046365fe

commit ace0b97f971549f85648facf01ef4542046365fe
Author:     Ilya Tumaykin <itumaykin@gmail.com>
AuthorDate: 2017-10-08 20:15:30 +0000
Commit:     Patrice Clement <monsieurp@gentoo.org>
CommitDate: 2017-10-08 22:01:46 +0000

    app-i18n/uchardet: disable flaky test.
    
    Closes: https://bugs.gentoo.org/631852
    Closes: https://github.com/gentoo/gentoo/pull/5890
    Package-Manager: Portage-2.3.11, Repoman-2.3.3

 app-i18n/uchardet/uchardet-0.0.6.ebuild | 4 ++++
 1 file changed, 4 insertions(+)
Comment 4 Patrice Clement gentoo-dev 2017-10-10 19:59:38 UTC
Hi Coacher

After chatting with Whissi in the #gentoo-proxy-maint channel, it appears this package actually doesn't run properly on the x86 platform. The test you are disabling is a clear indicator of that. I don't own x86 hardware but Whissi does and the package is broken on this arch he says.

What do we do? Should we mask this package on x86 for the time being?

Thanks!
Comment 5 Arfrever Frehtes Taifersar Arahesis 2017-10-11 15:30:21 UTC
(In reply to Larry the Git Cow from comment #3)

If test failure is specific to x86, then you could use:

if use x86; then
    rm test/th/tis-620.txt || die
fi
Comment 6 Thomas Deutschmann (RETIRED) gentoo-dev 2017-10-11 16:51:09 UTC
(In reply to Arfrever Frehtes Taifersar Arahesis from comment #5)
> (In reply to Larry the Git Cow from comment #3)
> 
> If test failure is specific to x86, then you could use:
> 
> if use x86; then
>     rm test/th/tis-620.txt || die
> fi
No. The test itself is working flawlessly according to upstream. However, it uncovered that the code itself doesn't do what you would expect on x86 when SSE support isn't available (see $URL).

So at the moment, the package doesn't work on every x86 system, therefor we cannot mark this package stable on x86. Just removing the test would hide the problem. But then we don't need arch teams and stabilization at all.
Comment 7 Coacher 2017-10-12 15:15:38 UTC
(In reply to Thomas Deutschmann from comment #6)
> (In reply to Arfrever Frehtes Taifersar Arahesis from comment #5)
> No. The test itself is working flawlessly according to upstream. However, it
> uncovered that the code itself doesn't do what you would expect on x86 when
> SSE support isn't available (see $URL).
Yes, because x86 is special with floating point numbers: https://gcc.gnu.org/wiki/FloatingPointMath#x86note

> So at the moment, the package doesn't work on every x86 system, therefor we
> cannot mark this package stable on x86. Just removing the test would hide
> the problem. But then we don't need arch teams and stabilization at all.
I don't have x86 hardware and I'm open to ideas from people who have/use it.
How x86 team suggests we mitigate this special behaviour of the architecture?
Comment 8 Thomas Deutschmann (RETIRED) gentoo-dev 2017-10-20 00:16:57 UTC
My proposal: https://github.com/gentoo/gentoo/pull/5993

But let's see what toolchain/QA will say.
Comment 9 Larry the Git Cow gentoo-dev 2017-11-19 14:32:37 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6a9d8fce1d916fc8590d654598648866978d4361

commit 6a9d8fce1d916fc8590d654598648866978d4361
Author:     Ilya Tumaykin <itumaykin@gmail.com>
AuthorDate: 2017-11-19 13:11:36 +0000
Commit:     David Seifert <soap@gentoo.org>
CommitDate: 2017-11-19 14:31:59 +0000

    app-i18n/uchardet: backport patch to enforce IEEE float precision
    
    Closes: https://bugs.gentoo.org/631852
    Closes: https://github.com/gentoo/gentoo/pull/6237
    Package-Manager: Portage-2.3.14, Repoman-2.3.6

 ...hardet-0.0.6-enforce-IEEE-float-precision.patch | 53 ++++++++++++++++++++++
 app-i18n/uchardet/uchardet-0.0.6-r1.ebuild         | 30 ++++++++++++
 2 files changed, 83 insertions(+)