GNU bug report logs - #63325
[PATCH] gnu: vulkan-loader: Only run tests on x86.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 63325 in the body.
You can then email your comments to 63325 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: vulkan-loader: Only run tests on x86.
Date: Sat,  6 May 2023 10:04:18 +0100

* gnu/packages/vulkan.scm (vulkan-loader)[arguments]: Only run tests on x86.
---
 gnu/packages/vulkan.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/vulkan.scm b/gnu/packages/vulkan.scm
index b0d968938b..f03c3dbc6f 100644
--- a/gnu/packages/vulkan.scm
+++ b/gnu/packages/vulkan.scm
@@ -228,8 +228,10 @@ (define-public vulkan-loader
     (build-system cmake-build-system)
     (arguments
      (list
-      #:tests? (not (or (%current-target-system)
-                        (target-riscv64?)))
+      ;; TODO: Some tests seem to assume running on x86,
+      ;; e.g. TryLoadWrongBinaries
+      #:tests? (and (target-x86?)
+                    (not (%current-target-system)))
       #:configure-flags
       #~(list (string-append "-DVULKAN_HEADERS_INSTALL_DIR="
                              (dirname (dirname

base-commit: 64086a4fa449a9f6d2f835fcdf5498222b309e3a
-- 
2.39.1

Message #8 received at 63325 <at> debbugs.gnu.org (full text, mbox):

From: Simon South <simon <at> simonsouth.net>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 63325 <at> debbugs.gnu.org
Subject: Re: [bug#63325] [PATCH] gnu: vulkan-loader: Only run tests on x86.
Date: Mon, 08 May 2023 10:54:34 -0400

Christopher Baines <mail <at> cbaines.net> writes:
> * gnu/packages/vulkan.scm (vulkan-loader)[arguments]: Only run tests on x86.

Christopher,

Disabling tests altogether seems extreme considering the only truly
x86-dependent ones appear to be five of the "TryLoadWrongBinaries" tests
(only these fail on AArch64), and this can perhaps be fixed by modifying
tests/framework/data/CMakeLists.txt to recreate the prebuilt binaries
included in the source package.

At any rate, I'd appreciate a day or two to see if I can come up with a
more nuanced patch that preserves the test suite on at least AArch64.

-- 
Simon South
simon <at> simonsouth.net

Message #11 received at 63325 <at> debbugs.gnu.org (full text, mbox):

From: Simon South <simon <at> simonsouth.net>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 63325 <at> debbugs.gnu.org
Subject: Re: [bug#63325] [PATCH] gnu: vulkan-loader: Only run tests on x86.
Date: Tue, 09 May 2023 16:53:04 -0400

Simon South <simon <at> simonsouth.net> writes:
> At any rate, I'd appreciate a day or two to see if I can come up with
> a more nuanced patch that preserves the test suite on at least
> AArch64.

To avoid hijacking this issue further I've opened one of my own, #63400,
with a patch that disables only the x86-specific tests on non-x86
platforms.

https://issues.guix.gnu.org/63400

Christopher, does this fix things for you as well?  For which
architecture are you building?

-- 
Simon South
simon <at> simonsouth.net

Message #14 received at 63325 <at> debbugs.gnu.org (full text, mbox):

From: Simon Tournier <zimon.toutoune <at> gmail.com>
To: Simon South <simon <at> simonsouth.net>, Christopher Baines <mail <at> cbaines.net>
Cc: 63325 <at> debbugs.gnu.org
Subject: Re: [bug#63325] [PATCH] gnu: vulkan-loader: Only run tests on x86.
Date: Thu, 11 May 2023 14:59:57 +0200

Hi,

On mar., 09 mai 2023 at 16:53, Simon South <simon <at> simonsouth.net> wrote:

> https://issues.guix.gnu.org/63400

I am in favor to close this submission and consider #63400.  Chris,
WDYT?

Cheers,
simon

Message #19 received at 63325-done <at> debbugs.gnu.org (full text, mbox):

From: Efraim Flashner <efraim <at> flashner.co.il>
To: Simon Tournier <zimon.toutoune <at> gmail.com>
Cc: 63325-done <at> debbugs.gnu.org, Christopher Baines <mail <at> cbaines.net>,
 Simon South <simon <at> simonsouth.net>
Subject: Re: [bug#63325] [PATCH] gnu: vulkan-loader: Only run tests on x86.
Date: Mon, 15 May 2023 22:04:37 +0300

[Message part 1 (text/plain, inline)]

On Thu, May 11, 2023 at 02:59:57PM +0200, Simon Tournier wrote:
> Hi,
> 
> On mar., 09 mai 2023 at 16:53, Simon South <simon <at> simonsouth.net> wrote:
> 
> > https://issues.guix.gnu.org/63400
> 
> I am in favor to close this submission and consider #63400.  Chris,
> WDYT?

Having come to basically the same patch (sorry Chris!) and having pushed
it, I'm going to close this patch and we can continue in 63400, which so
far has us skipping enough tests to pass on aarch64 but still fail on
other architectures.

-- 
Efraim Flashner   <efraim <at> flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[signature.asc (application/pgp-signature, inline)]

Message #22 received at 63325-done <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: Efraim Flashner <efraim <at> flashner.co.il>
Cc: 63325-done <at> debbugs.gnu.org
Subject: Re: [bug#63325] [PATCH] gnu: vulkan-loader: Only run tests on x86.
Date: Mon, 15 May 2023 21:16:57 +0100

[Message part 1 (text/plain, inline)]

Efraim Flashner <efraim <at> flashner.co.il> writes:

> [[PGP Signed Part:Good signature from 41AAE7DCCA3D8351 Efraim Flashner <efraim <at> flashner.co.il> (trust undefined) created at 2023-05-15T20:04:37+0100 using RSA]]
> On Thu, May 11, 2023 at 02:59:57PM +0200, Simon Tournier wrote:
>> Hi,
>> 
>> On mar., 09 mai 2023 at 16:53, Simon South <simon <at> simonsouth.net> wrote:
>> 
>> > https://issues.guix.gnu.org/63400
>> 
>> I am in favor to close this submission and consider #63400.  Chris,
>> WDYT?
>
> Having come to basically the same patch (sorry Chris!) and having pushed
> it, I'm going to close this patch and we can continue in 63400, which so
> far has us skipping enough tests to pass on aarch64 but still fail on
> other architectures.

No worries, I wasn't planning to merge this before looking at #63400
anyway.

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.