Subject: bug#72700: closed (Re: [PATCH] gnu: Remove an unnecessary test timeout for at-spi2-core)

From: [email-masked] (GNU bug Tracking System)
Date: Sun, 18 Aug 2024 22:38:02 +0000

Your bug report

#72700: [PATCH] gnu: Remove an unnecessary test timeout for at-spi2-core

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
72700: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72700
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [PATCH] gnu: Remove an unnecessary test timeout for at-spi2-core
From: Tobias Geerinckx-Rice <[email-masked]>
To: [email-masked]
Date: Mon, 19 Aug 2024 00:36:59 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Thanks!  Pushed as 58a839273d1e4fac2b3a0ec456aabdf82deaa124.

I tweaked the commit message to fit our norms, which mostly match 
the GNU ‘changelog’ style:

    gnu: at-spi2-core: Remove unnecessary test timeout.
    
    [Free-form blurb goes here, if you want, but more often than 
    not
     future readers are better served by a succinct code comment.]
    
    * gnu/packages/gtk.scm (at-spi2-core)[arguments]: Remove the
    ‘timeout’ argument from the test invocation.

Closing,

T G-R

Subject: [PATCH] gnu: Remove an unnecessary test timeout for at-spi2-core
From: Nikola <[email-masked]>
To: [email-masked]
Cc: Nikola <[email-masked]>
Date: Sun, 18 Aug 2024 13:59:45 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

aarch64 builds keep failing because of this timeout, so removing this
timeout should help
---
 gnu/packages/gtk.scm | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index 04183fa806..05041b57ab 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -856,6 +856,10 @@ (define-public at-spi2-core
                        (with-directory-excursion (string-append "../at-spi2-core-"
                                                                 #$version "")
                          (invoke "dbus-run-session" "--" "ci/run-registryd-tests.sh")
+                         ;; Remove test timeout which caused failed builds on
+                         ;; aarch64
+                         (substitute* "tests/atspi/meson.build"
+                           ((", timeout: [0-9]+") ""))
                          (substitute* "ci/run-tests.sh"
                            (("ps auxwww") "")   ;avoid a dependency on procps
                            (("meson test -C _build")
-- 
2.45.2