From: ludo@gnu.org (Ludovic Courtès)
To: Mathieu Lirzin <mthl@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] build: Add a scheme custom test driver using SRFI-64.
Date: Mon, 02 Nov 2015 09:59:28 +0100 [thread overview]
Message-ID: <87k2q0n5e7.fsf@gnu.org> (raw)
In-Reply-To: <87lhahcxgt.fsf@gnu.org> (Mathieu Lirzin's message of "Sun, 01 Nov 2015 20:48:34 +0100")
Mathieu Lirzin <mthl@gnu.org> skribis:
> ludo@gnu.org (Ludovic Courtès) writes:
[...]
>>> One caveat is that ‘tests/cpio.scm‘ is now failing.
>>
>> Why is that? Does it relate to this change?
>
> I didn't try to debug the problem but it has appeared when I started
> messing with redirection of the output/error ports. Here is the failure
> output.
>
> test-name: bit-identical to GNU cpio's output
> location: /home/mthl/src/gnu/guix/tests/cpio.scm:49
Is GNU cpio available in $PATH? Which version is that? Could you run
this test in ‘master’ to see if the problem shows up?
>>> Since this script is not intented to be exclusively used by Guix, I have
>>> used a generic copyright notice. I guess Guix is the best place to
>>> challenge and improve it, but IMO it will be better hosted elsewhere
>>> like in Gnulib. Opinions?
>>
>> I think we could start using it and testing it for a while in Guix, and
>> eventually submit it for inclusion in Gnulib once we are more confident.
>
> So you recommend to add ”this file is part of GNU Guix” and use “GNU
> Guix” instead of “this program” for now?
I think you can leave “this program” so nothing will need to be changed
when you move it elsewhere.
>> However, I’m unsure if we should push it now, or after the release. On
>> one hand, I’d rather avoid potentially disrupting changes like this
>> now. On the other hand, since it makes it easier (and different) to
>> report test failures, it’s nice.
>>
>> How confident are you? :-)
>
> IMHO we should wait after the next release in order to make this test
> driver more bullet proof.
Sounds reasonable.
Thanks!
Ludo’.
next prev parent reply other threads:[~2015-11-02 8:59 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-01 2:59 [PATCH] build: Add a scheme custom test driver using SRFI-64 Mathieu Lirzin
2015-11-01 17:13 ` Ludovic Courtès
2015-11-01 19:48 ` Mathieu Lirzin
2015-11-01 23:23 ` Mathieu Lirzin
2015-11-02 8:48 ` Ludovic Courtès
2015-11-02 8:59 ` Ludovic Courtès [this message]
2015-11-05 21:44 ` Mathieu Lirzin
2015-11-06 21:20 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k2q0n5e7.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=guix-devel@gnu.org \
--cc=mthl@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).