all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Chris Marusich <cmmarusich@gmail.com>
To: 47698@debbugs.gnu.org
Subject: bug#47698: [powerpc64le-linux] "check" package fails to build
Date: Thu, 03 Jun 2021 23:58:21 -0700	[thread overview]
Message-ID: <87eedit8tu.fsf@gmail.com> (raw)
In-Reply-To: <87wns1daic.fsf@gmail.com> (Chris Marusich's message of "Fri, 14 May 2021 00:45:15 -0700")


[-- Attachment #1.1: Type: text/plain, Size: 1270 bytes --]

Chris Marusich <cmmarusich@gmail.com> writes:

> [...] I've reported this issue upstream:
> https://github.com/libcheck/check/issues/333

Branden Archer replied in the above issue.  In short, the unreleased
upstream commit 4fbe702fa4f35bee8a90512f9f59d1441c4ae82e fixes this
issue on PPC platforms.  Here's what the commit does:

https://github.com/libcheck/check/commit/4fbe702fa4f35bee8a90512f9f59d1441c4ae82e.patch

  Adjust test suite for 106-bit long double precision

  On PowerPC architectures (ppc, ppc64el, powerp) 'long double' has a
  precision of 106-bit, compared to 80-bit precision on amd64.

  This leads to the test_ck_assert_(float|double|ldouble)_eq_tol succeed
  rather than fail as expected, cause 0.003-0.002 will be actually
  slightly bigger than 0.001 and not slightly smaller.

  Increase the change to the tolerance, so it will be on all architectures
  smaller than the difference of ~0.001 and the unit tests will fail as
  expected.

This commit was merged to the check repository's master branch after its
latest release (0.15.2).  It will be included in the next check release,
but until then, we will have to apply the fix as a patch to our check
package.  I've attached a patch that does this.

-- 
Chris

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-check-Fix-failing-tests-on-powerpc64le-linux.patch --]
[-- Type: text/x-patch, Size: 1978 bytes --]

From 7692295f970a292a3f3db31fc21d05efd97dcb25 Mon Sep 17 00:00:00 2001
From: Chris Marusich <cmmarusich@gmail.com>
Date: Thu, 3 Jun 2021 23:12:24 -0700
Subject: [PATCH] gnu: check: Fix failing tests on powerpc64le-linux.

* gnu/packages/check.scm (check)[source]: Apply unreleased upstream commit
4fbe702 as a patch.
---
 gnu/packages/check.scm | 22 +++++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/check.scm b/gnu/packages/check.scm
index 6641a0de58..069d4e05fc 100644
--- a/gnu/packages/check.scm
+++ b/gnu/packages/check.scm
@@ -146,7 +146,27 @@ like Jasmine or Mocha.")
                           version "/check-" version ".tar.gz"))
       (sha256
        (base32
-        "02m25y9m46pb6n46s51av62kpd936lkfv3b13kfpckgvmh5lxpm8"))))
+        "02m25y9m46pb6n46s51av62kpd936lkfv3b13kfpckgvmh5lxpm8"))
+      (patches
+       (list
+        ;; This patch fixes some tests that would otherwise fail on
+        ;; powerpc64le-linux.  Without this patch, the tests make certain
+        ;; assumptions about floating point number precision that are not true
+        ;; on that platform.
+        ;;
+        ;; TODO: Remove this patch when updating to the next check release,
+        ;; since it will be included there.  See:
+        ;; https://debbugs.gnu.org/cgi/bugreport.cgi?bug=47698
+        (origin
+          (method url-fetch)
+          (uri
+           (string-append "https://github.com/libcheck/check/commit/"
+                          "4fbe702fa4f35bee8a90512f9f59d1441c4ae82e.patch"))
+          (file-name (string-append name
+                                    "-fix-test-precision-for-ppc.patch"))
+          (sha256
+           (base32
+            "04qg1p9afdd6453k18qskazrvscysdcjz9j6w4i6p5x4xyma19v6")))))))
     (build-system gnu-build-system)
     (home-page "https://libcheck.github.io/check/")
     (synopsis "Unit test framework for C")
-- 
2.30.2


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 861 bytes --]

  reply	other threads:[~2021-06-04  7:16 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-11  1:00 bug#47698: [powerpc64le-linux] "check" package fails to build Chris Marusich
2021-04-12 19:10 ` Chris Marusich
2021-04-13 11:12   ` Efraim Flashner
2021-05-14  7:45 ` Chris Marusich
2021-06-04  6:58   ` Chris Marusich [this message]
2021-06-04  9:11     ` Efraim Flashner
2021-06-04  9:36     ` Efraim Flashner
2021-06-08  8:22       ` Efraim Flashner
2021-06-18 18:09         ` Chris Marusich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87eedit8tu.fsf@gmail.com \
    --to=cmmarusich@gmail.com \
    --cc=47698@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.