GNU bug report logs - #48319
[PATCH] Check if #:tests? is unconditionally #t; not acceptable when cross-compiling

Previous Next

Package: guix-patches;

Reported by: Maxime Devos <maximedevos <at> telenet.be>

Date: Sun, 9 May 2021 16:49:01 UTC

Severity: normal

Tags: patch

Done: Mathieu Othacehe <othacehe <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


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

From: Maxime Devos <maximedevos <at> telenet.be>
To: Mathieu Othacehe <othacehe <at> gnu.org>
Cc: 48319 <at> debbugs.gnu.org
Subject: Re: bug#48319: [PATCH] Check if #:tests? is unconditionally #t; not
 acceptable when cross-compiling
Date: Sat, 05 Jun 2021 11:56:25 +0200
[Message part 1 (text/plain, inline)]
Mathieu Othacehe schreef op vr 04-06-2021 om 14:50 [+0200]:
> Hello Maxime,
> 
> > +                          ;; TRANSLATORS: #:tests? and #t are Scheme constants
> > +                          ;; and must not be translated.
> > +                          (G_ "#:tests? does not need to be explicitly set to #t")
> > +                          #:field 'arguments))
> 
> This patch looks fine, and I remember fixing multiple packages failing
> to cross-compile because #:tests was hard-coded to #t.
> 
> I'm not sure about the warning message though. What about:
> 
> "#:tests? must not be explicitly set to #t"
> 
> as "does not need" implies it could be sometimes correct, whereas it is
> always a mistake.

The new warning message "#:tests? must not be explicitly set to #t"
seems reasonable to me. I'm currently working on other things though
(guix and other things), so don't expect a revised patch soon.

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

This bug report was last modified 3 years and 347 days ago.

Previous Next


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