guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#27599] [PATCH 1/2] gnu: Add cmdtest.


From: Kei Kebreau
Subject: [bug#27599] [PATCH 1/2] gnu: Add cmdtest.
Date: Fri, 07 Jul 2017 12:17:33 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)

Kei Kebreau <address@hidden> writes:

> Arun Isaac <address@hidden> writes:
>
>>> +(define-public cmdtest
>>> +  (package
>>> +    (name "cmdtest")
>>> +    (version "0.29")
>>> +    (source (origin
>>> +              (method url-fetch)
>>> +              (uri (string-append "http://git.liw.fi/cmdtest/snapshot/";
>>> +                                  name "-" version ".tar.gz"))
>>> +              (sha256
>>> +               (base32
>>> +                "1i6gi4yp4qqx1liax098c7nwdb24pghh11xqlrcs7lnhh079rqhb"))))
>>> +    (build-system python-build-system)
>>> +    (arguments
>>> +     `(#:python ,python-2))
>>
>> Could you try packaging the tests as well? The #:test-target should be
>> "check".
>>
>
> Thanks for the heads up here. I'm going to see what's wrong with the
> test suite now.
>

I can't figure out what's going wrong with the test suite. Perhaps
someone familiar with the Coverage Test Runner module in python can be
of assistance.

>>> +    (description
>>> + "@code{cmdtest} black box tests Unix command line tools.
>>> Roughly, it is
>>> +given a command line and input files, and the expected output, and
>>> it verifies
>>> +that the command line produces the expected output.  If not, it reports a
>>> +problem, and shows the differences.")
>>
>> The first sentence of the description should be a full sentence.
>
> That could be interpreted as a sentence, though I agree it's rough. I'll
> replace it in the next patch.

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]