Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
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
Doc Type: Bug Fix
Doc Text:
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.