GNU bug report logs - #47506
[PATCH] gnu: fiano-fmap: turn off tests to build program.

Previous Next

Package: guix;

Reported by: Adam Kandur <rndd <at> tuta.io>

Date: Tue, 30 Mar 2021 22:51:01 UTC

Severity: normal

Tags: patch

Merged with 45172

Done: Sharlatan Hellseher <sharlatanus <at> gmail.com>

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#47506; Package guix-patches. (Tue, 30 Mar 2021 22:51:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Adam Kandur <rndd <at> tuta.io>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Tue, 30 Mar 2021 22:51:02 GMT) Full text and rfc822 format available.

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

From: Adam Kandur <rndd <at> tuta.io>
To: Guix Patches <guix-patches <at> gnu.org>
Subject: [PATCH] gnu: fiano-fmap: turn off tests to build program.
Date: Wed, 31 Mar 2021 00:50:05 +0200 (CEST)
From b5a149a0ae68eb2692463f0d77f3da686ff23864 Mon Sep 17 00:00:00 2001
From: Adam Kandur <rndd <at> tuta.io>
Date: Wed, 31 Mar 2021 01:46:49 +0300
Subject: [PATCH] gnu: fiano-fmap: turn off tests to build program.

* gnu/packages/admin.scm (fiano-fmap): turn off tests to build program.
---
 gnu/packages/admin.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 292a4bc..14f7864 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -4431,7 +4431,9 @@ disk utilization, priority, username, state, and exit code.")
     (name "fiano-fmap")
     (arguments
      `(#:import-path "github.com/linuxboot/fiano/cmds/fmap"
-       #:unpack-path "github.com/linuxboot/fiano"))))
+       #:unpack-path "github.com/linuxboot/fiano"
+       ;; tests dont work
+       #:tests? #f))))
 
 (define-public novena-eeprom
   (package
-- 
2.30.1






Information forwarded to guix-patches <at> gnu.org:
bug#47506; Package guix-patches. (Wed, 31 Mar 2021 06:30:02 GMT) Full text and rfc822 format available.

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

From: Jack Hill <jackhill <at> jackhill.us>
To: Adam Kandur <rndd <at> tuta.io>
Cc: 47506 <at> debbugs.gnu.org
Subject: Re: [bug#47506] [PATCH] gnu: fiano-fmap: turn off tests to build
 program.
Date: Wed, 31 Mar 2021 02:28:55 -0400 (EDT)
[Message part 1 (text/plain, inline)]
Hi,

Thanks for your interest in fiano-fmap!

On Wed, 31 Mar 2021, Adam Kandur via Guix-patches via wrote:

>
> From b5a149a0ae68eb2692463f0d77f3da686ff23864 Mon Sep 17 00:00:00 2001
> From: Adam Kandur <rndd <at> tuta.io>
> Date: Wed, 31 Mar 2021 01:46:49 +0300
> Subject: [PATCH] gnu: fiano-fmap: turn off tests to build program.
>
> * gnu/packages/admin.scm (fiano-fmap): turn off tests to build program.
> ---
>  gnu/packages/admin.scm | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
> index 292a4bc..14f7864 100644
> --- a/gnu/packages/admin.scm
> +++ b/gnu/packages/admin.scm
> @@ -4431,7 +4431,9 @@ disk utilization, priority, username, state, and exit code.")
>      (name "fiano-fmap")
>      (arguments
>       `(#:import-path "github.com/linuxboot/fiano/cmds/fmap"
> -       #:unpack-path "github.com/linuxboot/fiano"))))
> +       #:unpack-path "github.com/linuxboot/fiano"
> +       ;; tests dont work
> +       #:tests? #f))))
>  
>  (define-public novena-eeprom
>    (package
>

When I first noticed the test failure in #45172 [0] it wasn't clear to me 
that the test failure was harmless rather than exhibiting an actual problem 
in our fiano-fmap package. Can you confirm that the package works as 
expected? I also wonder if we could make a smaller change and just disable 
the problematic test rather than the whole suite.

I've also noticed that fiano-fmap has a release candidate for version 
6.0.0. I wonder if that version fixes the problem (I may have to give it a 
try, although this one may come back to needing improvements in the 
go-build-system).

[0] https://issues.guix.gnu.org/45172

Best,
Jack

bug reassigned from package 'guix-patches' to 'guix'. Request was from Jack Hill <jackhill <at> jackhill.us> to control <at> debbugs.gnu.org. (Wed, 31 Mar 2021 14:44:02 GMT) Full text and rfc822 format available.

Merged 45172 47506. Request was from Jack Hill <jackhill <at> jackhill.us> to control <at> debbugs.gnu.org. (Wed, 31 Mar 2021 14:44:02 GMT) Full text and rfc822 format available.

Information forwarded to bug-guix <at> gnu.org:
bug#47506; Package guix. (Thu, 04 Apr 2024 16:47:02 GMT) Full text and rfc822 format available.

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

From: Sharlatan Hellseher <sharlatanus <at> gmail.com>
To: 47506-done <at> debbugs.gnu.org
Subject: [PATCH] gnu: fiano-fmap: turn off tests to build program.
Date: Thu, 04 Apr 2024 17:46:03 +0100
[Message part 1 (text/plain, inline)]
Hi,

Closing this issue as resolved in <https://issues.guix.gnu.org/70123>.

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

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Fri, 03 May 2024 11:24:05 GMT) Full text and rfc822 format available.

This bug report was last modified today.

Previous Next


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