Bug 1083145 - findutils ppc64le tests failed (test-isinf, test-isnanl-nolibm)
Summary: findutils ppc64le tests failed (test-isinf, test-isnanl-nolibm)
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: findutils
Version: 21
Hardware: ppc64le
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Kamil Dudka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F-ExcludeArch-ppc64le, PPC64LETracker
TreeView+ depends on / blocked
 
Reported: 2014-04-01 14:38 UTC by Menanteau Guy
Modified: 2015-01-20 12:59 UTC (History)
4 users (show)

Fixed In Version: findutils-4.5.12-4.fc21
Clone Of:
Environment:
Last Closed: 2015-01-20 12:59:39 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Menanteau Guy 2014-04-01 14:38:40 UTC
Description of problem:
tests failed during build (make check) of findutils in ppc64le arch

Version-Release number of selected component (if applicable):
findutils-4.5.12-2.fc21

How reproducible:
rpmbuild --target ppc64le

Actual results:
../build-aux/test-driver: line 107: 25009 Aborted                 "$@" > $log_file 2>&1
FAIL: test-isinf
PASS: test-isnand-nolibm
PASS: test-isnanf-nolibm
../build-aux/test-driver: line 107: 25027 Aborted                 "$@" > $log_file 2>&1
FAIL: test-isnanl-nolibm

Additional info:
floating-point test cases do not work correctly with the PowerPC long double ("double double")format.  These tests "accidentally" succeeded anyway in the big-endian variant, but are now actually failing in little-endian mode.

A patch to fix the tests for ppc64le has been submitted to upstream
to the bug-gnulib mailing list.
http://lists.gnu.org/archive/html/bug-gnulib/2013-12/msg00107.html

note that this bug affects packages like gnulib, including coreutils, findutils, grep, and libunistring.

Comment 1 Kamil Dudka 2014-04-02 13:01:26 UTC
Are there any concerns that prevent the patch from being applied upstream?

Comment 2 Menanteau Guy 2014-04-03 10:16:57 UTC
This patch have been written originally by Ulrich Weigand (see mail archive url I put in description) and it concerns sevreal packages. Finally I didn't found what have been done in upstream for findutils. 
In my point of view, it fixes failing tests and I didn't see side effects but I am not the best person to validate it in upstream.

Comment 3 Kamil Dudka 2014-04-03 10:56:35 UTC
It needs to be accepted by _gnulib_ upstream.  Then findutils will automatically get the fix on a next import of gnulib...

Comment 4 Dan Horák 2014-06-02 10:49:38 UTC
the gnulib fix is applied in upstream as
http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commitdiff;h=e962c07c0

Kamil, please apply the fix also to findutils package, it is currently blocking the merge of our bootstrap ppc64 little endian koji with the existing ppc koji.

Comment 5 Kamil Dudka 2014-06-02 11:46:25 UTC
fixed in indutils-4.5.12-4.fc21

Thank you for getting the patch upstream!

Comment 6 Kamil Dudka 2015-01-20 12:59:39 UTC
Already fixed in Fedora 21, closing...


Note You need to log in before you can comment on or make changes to this bug.