wabt: fix builds on !x86_64 and update to 1.0.34

  • Done
  • quality assurance status badge
Details
One participant
  • Vagrant Cascadian
Owner
unassigned
Submitted by
Vagrant Cascadian
Severity
normal

Debbugs page

Vagrant Cascadian wrote 1 years ago
fix builds on !x86_64 and update to 1.0.34
(address . guix-patches@gnu.org)
871q9u8e03.fsf@wireframe
The attached patch updates wabt to 1.0.34 and also disables tests on
non-x86_64 architectures.

Since the update to wabt 1.0.32 in commit
9d474e105977e46d1ffd9afe794d26882a66c7a0, tests were also enabled... but
it has failed to build architectures other than x86_64 ever since...

Because of this, diffoscope and reprotest have not been able to build,
as diffoscope has wabt in native-inputs.

Thanks to ulfvonbelow via #guix IRC for the simple suggestion of how to
disable tests!

Another option would be to selectively disable tests, though
historically there have been a number of relevent outstanding issues
related to tests on !x86_64, which might turn into a few rounds of
whack-a-mole every time wabt is updated:



live well,
vagrant
From a378a8b7b23c2be9f8374d28fc1f89134a9fd272 Mon Sep 17 00:00:00 2001
From: Vagrant Cascadian <vagrant@reproducible-builds.org>
Date: Fri, 2 Feb 2024 15:32:04 -0800
Subject: [PATCH] gnu: wabt: Update to 1.0.34.

* gnu/packages/web.scm (wabt): Update to 1.0.34.
[arguments]: Disable tests on !x86_64.
---
gnu/packages/web.scm | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

Toggle diff (34 lines)
diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index 1a97dd8dec..1eefb1fde7 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -1638,7 +1638,7 @@ (define-public libwebsockets
(define-public wabt
(package
(name "wabt")
- (version "1.0.32")
+ (version "1.0.34")
(source
(origin
(method git-fetch)
@@ -1648,13 +1648,15 @@ (define-public wabt
(recursive? #true)))
(file-name (git-file-name name version))
(sha256
- (base32 "0m124r8v9c0hxiaa4iy7ch4ng8msnirbc2vb702gbdjhvgzyrcwh"))
+ (base32 "1vxvc34b7a7lkrmzdb5cjv0b54vhiyr33sy0i2ps5jrmg5rqqmia"))
(modules '((guix build utils)))
(snippet
'(delete-file-recursively "third_party/gtest/"))))
(build-system cmake-build-system)
(arguments
(list
+ ;; Tests on non-x86_64 architectures are not well supported upstream.
+ #:tests? (target-x86-64?)
#:test-target "run-tests"
#:configure-flags '(list "-DUSE_SYSTEM_GTEST=ON")
#:phases

base-commit: f45d05133472a9da13eae20ba4a676c696682c90
--
2.39.2
-----BEGIN PGP SIGNATURE-----

iHUEARYKAB0WIQRlgHNhO/zFx+LkXUXcUY/If5cWqgUCZb2oHAAKCRDcUY/If5cW
qhTAAQCfturehjXuNqmNaNFLsGbPAJeWzEMoA+5erVPECotcFAEAoGmQ43ntSlAh
u3pbViNZ/SWW6s1lrbA2kT+D3Vg/uwg=
=0p4i
-----END PGP SIGNATURE-----

Vagrant Cascadian wrote 1 years ago
(address . control@debbugs.gnu.org)
87y1bxqihg.fsf@wireframe
retitle 68903 wabt: fix builds on !x86_64 and update to 1.0.34
thanks

fix title, oops!

live well,
vagrant
Vagrant Cascadian wrote 1 years ago
(address . 68903-done@debbugs.gnu.org)
87o7cp2ttg.fsf@wireframe
On 2024-02-02, Vagrant Cascadian wrote:
Toggle quote (10 lines)
> The attached patch updates wabt to 1.0.34 and also disables tests on
> non-x86_64 architectures.
>
> Since the update to wabt 1.0.32 in commit
> 9d474e105977e46d1ffd9afe794d26882a66c7a0, tests were also enabled... but
> it has failed to build architectures other than x86_64 ever since...
>
> Because of this, diffoscope and reprotest have not been able to build,
> as diffoscope has wabt in native-inputs.

Pushed to master as 8ffc9233681863408034c8e371c5cfd3591545d2.

Looking forward to diffoscope on non-x86_64 again!

live well,
vagrant
-----BEGIN PGP SIGNATURE-----

iHUEARYKAB0WIQRlgHNhO/zFx+LkXUXcUY/If5cWqgUCZcadiwAKCRDcUY/If5cW
qiH6AQCcGDpQPkDfS7zZe8KCenMgZfnMp8ZBvLqVIhM+1n3uSgD9E8jiZ0AezqXr
O3/0XpBbspLb3RO93mSn3uAilwjEIAg=
=k8Fe
-----END PGP SIGNATURE-----

Closed
?
Your comment

This issue is archived.

To comment on this conversation send an email to 68903@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 68903
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help