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

From: Kei Kebreau <[email-masked]>
Date: Thu, 06 Jul 2017 21:02:44 -0400

Arun Isaac <[email-masked]> 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.

>> +    (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.