all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#68484] [PATCH] gnu: guile-fibers: Disable failing tests on aarch64.
@ 2024-01-15 19:57 Roman Scherer
  2024-01-17  9:56 ` Mathieu Othacehe
  2024-01-17 19:48 ` [bug#68484] [PATCH v2] " Roman Scherer
  0 siblings, 2 replies; 6+ messages in thread
From: Roman Scherer @ 2024-01-15 19:57 UTC (permalink / raw)
  To: 68484; +Cc: Roman Scherer

* gnu/packages/guile-xyz.scm (guile-fibers): Disable failing tests on aarch64.

Change-Id: Id05b516a659f641cc7ef01efdaf6cd1f2c735800
---
 gnu/packages/guile-xyz.scm | 31 +++++++++++++++++++++----------
 1 file changed, 21 insertions(+), 10 deletions(-)

diff --git a/gnu/packages/guile-xyz.scm b/gnu/packages/guile-xyz.scm
index 77be632cc3..9b0f27016d 100644
--- a/gnu/packages/guile-xyz.scm
+++ b/gnu/packages/guile-xyz.scm
@@ -824,16 +824,27 @@ (define-public guile-fibers
            #~(list "GUILE_AUTO_COMPILE=0")
            #:phases
            (if (target-x86-64?)
-             #~%standard-phases
-             #~(modify-phases %standard-phases
-                 (add-before 'check 'disable-some-tests
-                   (lambda _
-                     ;; This test can take more than an hour on some systems.
-                     (substitute* "tests/basic.scm"
-                       ((".*spawn-fiber loop-to-1e4.*") ""))
-                     ;; These tests can take more than an hour and/or segfault.
-                     (substitute* "Makefile"
-                       (("tests/speedup.scm") ""))))))))
+               #~%standard-phases
+               #~(modify-phases %standard-phases
+                   (add-before 'check 'disable-some-tests
+                     (lambda _
+                       (substitute* "tests/basic.scm"
+                         ;; This test can take more than an hour on some systems.
+                         ((".*spawn-fiber loop-to-1e4.*") "")
+                         ;; This test has issues on aarch64 systems. It passes
+                         ;; on an Apple M1, but takes a very long time on a
+                         ;; Hetzner aarch64 VM.
+                         ((".*spawn-fiber-chain 5000000.*") ""))
+
+                       (substitute* "tests/channels.scm"
+                         ;; This test has issues on aarch64 systems. It passes
+                         ;; on an Apple M1, but raises exceptions on a Hetzner
+                         ;; aarch64 VM.
+                         ((".*assert-run-fibers-terminates .*pingpong.*") ""))
+
+                       ;; These tests can take more than an hour and/or segfault.
+                       (substitute* "Makefile"
+                         (("tests/speedup.scm") ""))))))))
     (native-inputs
      (list texinfo pkg-config autoconf-2.71 automake libtool
            guile-3.0            ;for 'guild compile

base-commit: 162d6a2fdd6af13272967c77347a54934ecb45e6
-- 
2.41.0





^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#68484] [PATCH] gnu: guile-fibers: Disable failing tests on aarch64.
  2024-01-15 19:57 [bug#68484] [PATCH] gnu: guile-fibers: Disable failing tests on aarch64 Roman Scherer
@ 2024-01-17  9:56 ` Mathieu Othacehe
  2024-01-17 19:46   ` Roman Scherer
  2024-01-17 19:48 ` [bug#68484] [PATCH v2] " Roman Scherer
  1 sibling, 1 reply; 6+ messages in thread
From: Mathieu Othacehe @ 2024-01-17  9:56 UTC (permalink / raw)
  To: Roman Scherer; +Cc: 68484


Hey,

> +                       (substitute* "tests/channels.scm"
> +                         ;; This test has issues on aarch64 systems. It passes
> +                         ;; on an Apple M1, but raises exceptions on a Hetzner
> +                         ;; aarch64 VM.
> +                         ((".*assert-run-fibers-terminates .*pingpong.*") ""))

Maybe we should only disable that one if we are targeting an aarch64
system / target?

Thanks,

Mathieu




^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#68484] [PATCH] gnu: guile-fibers: Disable failing tests on aarch64.
  2024-01-17  9:56 ` Mathieu Othacehe
@ 2024-01-17 19:46   ` Roman Scherer
  0 siblings, 0 replies; 6+ messages in thread
From: Roman Scherer @ 2024-01-17 19:46 UTC (permalink / raw)
  To: Mathieu Othacehe; +Cc: 68484

[-- Attachment #1: Type: text/plain, Size: 625 bytes --]


Hi Mathieu,

thanks for taking a look. Yes, you are right. I'm going to send another
patch.

Roman

Mathieu Othacehe <othacehe@gnu.org> writes:

> Hey,
>
>> +                       (substitute* "tests/channels.scm"
>> +                         ;; This test has issues on aarch64 systems. It passes
>> +                         ;; on an Apple M1, but raises exceptions on a Hetzner
>> +                         ;; aarch64 VM.
>> +                         ((".*assert-run-fibers-terminates .*pingpong.*") ""))
>
> Maybe we should only disable that one if we are targeting an aarch64
> system / target?
>
> Thanks,
>
> Mathieu

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#68484] [PATCH v2] gnu: guile-fibers: Disable failing tests on aarch64.
  2024-01-15 19:57 [bug#68484] [PATCH] gnu: guile-fibers: Disable failing tests on aarch64 Roman Scherer
  2024-01-17  9:56 ` Mathieu Othacehe
@ 2024-01-17 19:48 ` Roman Scherer
  2024-01-24 22:30   ` bug#68484: " Ludovic Courtès
  1 sibling, 1 reply; 6+ messages in thread
From: Roman Scherer @ 2024-01-17 19:48 UTC (permalink / raw)
  To: 68484; +Cc: Roman Scherer

* gnu/packages/guile-xyz.scm (guile-fibers): Disable failing tests on aarch64.

Change-Id: Id05b516a659f641cc7ef01efdaf6cd1f2c735800
---
 gnu/packages/guile-xyz.scm | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/guile-xyz.scm b/gnu/packages/guile-xyz.scm
index 77be632cc3..1db0984a04 100644
--- a/gnu/packages/guile-xyz.scm
+++ b/gnu/packages/guile-xyz.scm
@@ -823,17 +823,30 @@ (define-public guile-fibers
      (list #:make-flags
            #~(list "GUILE_AUTO_COMPILE=0")
            #:phases
-           (if (target-x86-64?)
-             #~%standard-phases
-             #~(modify-phases %standard-phases
-                 (add-before 'check 'disable-some-tests
-                   (lambda _
+           #~(modify-phases %standard-phases
+               (add-before 'check 'disable-some-tests
+                 (lambda _
+                   (unless #$(target-x86-64?)
                      ;; This test can take more than an hour on some systems.
                      (substitute* "tests/basic.scm"
                        ((".*spawn-fiber loop-to-1e4.*") ""))
+
                      ;; These tests can take more than an hour and/or segfault.
                      (substitute* "Makefile"
-                       (("tests/speedup.scm") ""))))))))
+                       (("tests/speedup.scm") "")))
+
+                   (when #$(target-aarch64?)
+                     ;; This test has issues on aarch64 systems. It passes on
+                     ;; an Apple M1, but takes a very long time on a Hetzner
+                     ;; aarch64 VM.
+                     (substitute* "tests/basic.scm"
+                       ((".*spawn-fiber-chain 5000000.*") ""))
+
+                     ;; This test has issues on aarch64 systems. It passes on
+                     ;; an Apple M1, but raises exceptions on a Hetzner
+                     ;; aarch64 VM.
+                     (substitute* "tests/channels.scm"
+                       ((".*assert-run-fibers-terminates .*pingpong.*") ""))))))))
     (native-inputs
      (list texinfo pkg-config autoconf-2.71 automake libtool
            guile-3.0            ;for 'guild compile

base-commit: 162d6a2fdd6af13272967c77347a54934ecb45e6
-- 
2.41.0





^ permalink raw reply related	[flat|nested] 6+ messages in thread

* bug#68484: [PATCH v2] gnu: guile-fibers: Disable failing tests on aarch64.
  2024-01-17 19:48 ` [bug#68484] [PATCH v2] " Roman Scherer
@ 2024-01-24 22:30   ` Ludovic Courtès
  2024-01-24 22:45     ` [bug#68484] " Roman Scherer
  0 siblings, 1 reply; 6+ messages in thread
From: Ludovic Courtès @ 2024-01-24 22:30 UTC (permalink / raw)
  To: Roman Scherer; +Cc: 68484-done

Hi Roman,

Roman Scherer <roman@burningswell.com> skribis:

> * gnu/packages/guile-xyz.scm (guile-fibers): Disable failing tests on aarch64.
>
> Change-Id: Id05b516a659f641cc7ef01efdaf6cd1f2c735800

I pushed a variant of this patch as
8bee6bb9aaaf35c36fe325675d1eb2daebd69c25, which avoids a rebuild on
x86_64 and makes things somewhat clearer IMO.

Thank you,
Ludo’.




^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#68484] [PATCH v2] gnu: guile-fibers: Disable failing tests on aarch64.
  2024-01-24 22:30   ` bug#68484: " Ludovic Courtès
@ 2024-01-24 22:45     ` Roman Scherer
  0 siblings, 0 replies; 6+ messages in thread
From: Roman Scherer @ 2024-01-24 22:45 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 68484-done

[-- Attachment #1: Type: text/plain, Size: 507 bytes --]

Perfect. Thank you.

On Wed, Jan 24, 2024, 23:30 Ludovic Courtès <ludo@gnu.org> wrote:

> Hi Roman,
>
> Roman Scherer <roman@burningswell.com> skribis:
>
> > * gnu/packages/guile-xyz.scm (guile-fibers): Disable failing tests on
> aarch64.
> >
> > Change-Id: Id05b516a659f641cc7ef01efdaf6cd1f2c735800
>
> I pushed a variant of this patch as
> 8bee6bb9aaaf35c36fe325675d1eb2daebd69c25, which avoids a rebuild on
> x86_64 and makes things somewhat clearer IMO.
>
> Thank you,
> Ludo’.
>

[-- Attachment #2: Type: text/html, Size: 875 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2024-01-24 22:46 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-15 19:57 [bug#68484] [PATCH] gnu: guile-fibers: Disable failing tests on aarch64 Roman Scherer
2024-01-17  9:56 ` Mathieu Othacehe
2024-01-17 19:46   ` Roman Scherer
2024-01-17 19:48 ` [bug#68484] [PATCH v2] " Roman Scherer
2024-01-24 22:30   ` bug#68484: " Ludovic Courtès
2024-01-24 22:45     ` [bug#68484] " Roman Scherer

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.