guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] gnu: sdcv: Add new variable.


From: Ludovic Courtès
Subject: Re: [PATCH] gnu: sdcv: Add new variable.
Date: Wed, 18 Mar 2015 09:39:49 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux)

Tomáš Čech <address@hidden> skribis:

> * gnu/packages/dictionaries.scm (sdcv): New variable.

You pushed it already but I have some comments anyway:

> +(define-public sdcv
> +  (package
> +    (name "sdcv")
> +    (version "0.5.0-beta4")

The policy is to provide only stable versions, unless there’s a very
good reason to do otherwise.  Could we use the previous version until
0.5.0 is out?

> +    (synopsis "Command line variant of StarDict")

Could you change it to be self-contained–i.e., without referring to
StarDict (which I don’t know, and perhaps is not very well known.)

> +    (description
> +     "Sdcv is command line dictionary utility, which supports StarDict 
> dictinary
> +format.")

Typo: “dictionary”.

Please expound, explaining what it does concretely.  Perhaps get
inspiration from the README or web page.

TIA,
Ludo’.



reply via email to

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