From: Mathieu Lirzin <mthl@gnu.org>
To: Eric Bavier <ericbavier@openmailbox.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 0/5] Automake custom test driver using SRFI-64.
Date: Thu, 24 Mar 2016 18:02:26 +0100 [thread overview]
Message-ID: <87r3ezu7vh.fsf@gnu.org> (raw)
In-Reply-To: <20160323231545.5eb7e2c0@openmailbox.org> (Eric Bavier's message of "Wed, 23 Mar 2016 23:15:45 -0500")
Eric Bavier <ericbavier@openmailbox.org> writes:
> In addition to the test status, I get messages like this when running
> `make check`:
>
> -----
> ;;; note: source file /home/bavier/projects/guix/srfi/srfi-64.scm
> ;;; newer than compiled /gnu/store/4r7iv7lz7vbcvzdjilqpiflzkh8nff3b-profile/lib/guile/2.0/ccache/srfi/srfi-64.go
> PASS: tests/elpa.scm
> ./test-env: line 1: 19815 Terminated "/home/bavier/projects/guix/build/master/pre-inst-env" "/home/bavier/projects/guix/build/master/guix-daemon" --disable-chroot --substitute-urls="$GUIX_BINARY_SUBSTITUTE_URL" 2> /dev/null
> -----
>
> It seems that we don't compile any of the srfi/srfi-* source?
If you have a recent Guile version (>= 2.0.9) SRFI-64 is already
included and should be used without falling back to Guix embedded copy.
I have only tested with Guile 2.0.11 (from Debian) so I guess this
problem is appearing when using Guile from Guix.
Maybe 'Makefile.am' should include something like:
--8<---------------cut here---------------start------------->8---
if !HAVE_SRFI_64
GOBJECTS += srfi/srfi-64.go srfi/srfi-64.upstream.go
endif
--8<---------------cut here---------------end--------------->8---
with some m4 magic to define AM_CONDITIONAL([HAVE_SRFI_64], ... ) when
srfi-64 is found in Guile. :)
> I don't recall seeing the "Terminated" messages before (anyone else
> see this?), but this patch to test-env.in seems to help:
>
> --- a/test-env.in
> +++ b/test-env.in
> @@ -100,7 +100,8 @@ then
> --substitute-urls="$GUIX_BINARY_SUBSTITUTE_URL" 2>/dev/null &
>
> daemon_pid=$!
> - trap "kill $daemon_pid ; rm -rf $NIX_STATE_DIR" EXIT
> + trap "kill $daemon_pid ; wait $daemon_pid 2>/dev/null ;
> + rm -rf $NIX_STATE_DIR" EXIT
> fi
I don't have the "terminated" message on my system. Does it appear for
all the tests? If not, can you identify what tests are creating those
messages? (by launching ‘make check TESTS="tests/suspect.scm"’).
> The result output seems more informative. The counts include
> individual tests within each tests/*.scm, correct?
Yes, every individual test case is included in the summary.
> ============================================================================
> Testsuite summary for GNU Guix 0.9.1
> ============================================================================
> # TOTAL: 677
> # PASS: 671
> # SKIP: 6
> # XFAIL: 0
> # FAIL: 0
> # XPASS: 0
> # ERROR: 0
> ============================================================================
>
> I notice in the test-suite.log that logs are included from tests that
> are skipped. This seems appropriate, but it appears to also include
> PASS output from other tests in that suite (e.g. all of tests/gexp.log
> is included since two of the 49 tests are skipp). Is it possible to
> include output from only the skipped tests? I'm not familiar for the
> trs spec.
It is not possible unfortunately. The '.trs' files have only a
':copy-in-global-log:' field which defines if the corresponding '.log'
file is copied verbatim in test-suite.log or not.
> Thanks for this work. I had considered reworking the test suites with
> Autoconf's AutoTest for better unity, but I think what you've done here
> is quite a bit more elegant with less disruption.
I should learn how to use AutoTest. Thanks for your review and your
kind words.
--
Mathieu Lirzin
next prev parent reply other threads:[~2016-03-24 17:02 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-23 23:38 [PATCH 0/5] Automake custom test driver using SRFI-64 Mathieu Lirzin
2016-03-23 23:38 ` [PATCH 1/5] tests: Silence %cpio-program Mathieu Lirzin
2016-03-23 23:38 ` [PATCH 2/5] build: Add a Guile custom test driver using SRFI-64 Mathieu Lirzin
2016-03-23 23:38 ` [PATCH 3/5] tests: Silence guix-daemon Mathieu Lirzin
2016-03-23 23:38 ` [PATCH 4/5] tests: Silence %have-nix-hash? Mathieu Lirzin
2016-03-23 23:38 ` [PATCH 5/5] tests: Silence tar Mathieu Lirzin
2016-03-24 4:15 ` [PATCH 0/5] Automake custom test driver using SRFI-64 Eric Bavier
2016-03-24 17:02 ` Mathieu Lirzin [this message]
2016-03-27 21:39 ` Ludovic Courtès
2016-03-27 21:47 ` Mathieu Lirzin
2016-04-01 20:51 ` Ludovic Courtès
2016-04-02 10:59 ` Mathieu Lirzin
2016-04-02 13:22 ` Ludovic Courtès
2016-04-02 14:25 ` Mathieu Lirzin
2016-04-03 12:33 ` Mathieu Lirzin
2016-04-03 21:30 ` Ludovic Courtès
2016-04-01 21:45 ` 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=87r3ezu7vh.fsf@gnu.org \
--to=mthl@gnu.org \
--cc=ericbavier@openmailbox.org \
--cc=guix-devel@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).