civodul pushed a commit to branch master
in repository guix.

commit 72e586fcae78e467d01e2add09c1db26be6bfa93
Author: Timotej Lazar <timotej.la...@araneo.si>
AuthorDate: Tue Jun 25 15:14:16 2024 +0200

    gnu: gtk: Fix build on aarch64.
    
    * gnu/packages/gtk.scm (gtk)[arguments]<#:test-options>: Rename failing
    test suite to match upstream change on aarch64-linux.
    
    Change-Id: Ibdf03a94e71f72283a093cba6d5a328004defa5c
    Co-authored-by: Ludovic Courtès <l...@gnu.org>
---
 gnu/packages/gtk.scm | 36 +++++++++++++++++++++++-------------
 1 file changed, 23 insertions(+), 13 deletions(-)

diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index fb10351849..e5da382a12 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -1171,19 +1171,29 @@ application suites.")
                 '("-Dgtk_doc=false")
                 '("-Dgtk_doc=true"))
          "-Dman-pages=true")
-      #:test-options '(list "--setup=x11" ;defaults to wayland
-                            ;; Use the same test options as upstream uses for
-                            ;; their CI.
-                            "--suite=gtk"
-                            "--no-suite=failing"
-                            "--no-suite=flaky"
-                            "--no-suite=headless" ; requires mutter…
-                            "--no-suite=gsk-compare-broadway"
-                            ;; These seem to fail on aarch64, and Debian has
-                            ;; also disabled these, see:
-                            ;; 
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1050075
-                            "--no-suite=wayland_failing"
-                            "--no-suite=wayland_gles_failing")
+      #:test-options #~(list "--setup=x11" ;defaults to wayland
+                             ;; Use the same test options as upstream uses for
+                             ;; their CI.
+                             "--suite=gtk"
+                             "--no-suite=failing"
+                             "--no-suite=flaky"
+                             "--no-suite=headless" ; requires mutter…
+                             "--no-suite=gsk-compare-broadway"
+                             ;; These seem to fail on aarch64, and Debian has
+                             ;; also disabled these, see:
+                             ;; 
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1050075
+                             "--no-suite=wayland_failing"
+
+                             ;; The correct name of the test that fails on
+                             ;; AArch64 is 'wayland_gles2_failing':
+                             ;; 
<https://gitlab.gnome.org/GNOME/gtk/-/commit/b916c4dac1075572>.
+                             ;; The second arm of the 'if' is kept to avoid a
+                             ;; rebuild on other architectures.  TODO: Remove
+                             ;; on the next rebuild cycle.
+                             #$(if (and (not (%current-target-system))
+                                        (string=? (%current-system) 
"aarch64-linux"))
+                                   "--no-suite=wayland_gles2_failing"
+                                   "--no-suite=wayland_gles_failing"))
       #:phases
       #~(modify-phases %standard-phases
           (add-after 'unpack 'generate-gdk-pixbuf-loaders-cache-file

Reply via email to