Subject: [bug#47402] gnu: Add emacs-sdcv

From: Nicolas Goaziou <[email-masked]>
Date: Fri, 26 Mar 2021 15:03:27 +0100

Hello,

qblade via Guix-patches via <[email-masked]> writes:

> I submitted two patches, emacs-showtip is a dependency of emacs-sdcv.

Thank you. Some comments follow.

> +(define-public emacs-showtip
> +  (package
> +    (name "emacs-showtip")
> +    (version "20090830.1040")

This is not the correct version number. Main file indicates "0.01".

> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/emacsorphanage/showtip")
> +             (commit
> +              "930da302809a4257e8d69425455b29e1cc91949b")))
> +       (file-name (git-file-name name version))
> +       (sha256
> +        (base32
> +         "01zak0zhha6dp7a2hm28d065gjnc462iwpsfyxhbxgfzcdlicqc7"))))
> +    (build-system emacs-build-system)
> +    (home-page "https://github.com/emacsorphanage/showtip")
> +    (synopsis "Show tip at cursor")
> +    (description
> +     "This library export one function for elisp programer to show
> +tooltip near the cursor not the mouse.")

export -> exports or even, provides

I would also drop the "for Elisp programmer" part.

> +(define-public emacs-sdcv
> +  (package
> +    (name "emacs-sdcv")
> +    (version "20190610.732")

The correct version number seems to be 1.5.2 on the GitHub repository,
and 2.9 on emacswiki. However, using Emacswiki as a source is not an
option, since there is no archival for outdated releases there, AFAICT.

Are we stuck with an old release?

> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/stardiviner/sdcv.el")
> +             (commit
> +              "943ae3e90cc9a0a88a37cc710acd7424fd4defc4")))

Since you're targetting a specific commit, you need to add a commit and
a revision binding at the top of the definition. See, e.g.,
emacs-vterm-toggle.

> +       (file-name (git-file-name name version))
> +       (sha256
> +        (base32
> +         "0i1ylvw7p46pkf3yyyzcdmdhsspzymnnnvx8s0i7vynngr5x0vzh"))))
> +    (build-system emacs-build-system)
> +    (propagated-inputs
> +     `(("emacs-popup" ,emacs-popup)
> +       ("emacs-showtip" ,emacs-showtip)
> +       ("emacs-pos-tip" ,emacs-pos-tip)))
> +    (home-page
> +     "https://www.emacswiki.org/emacs/download/sdcv.el")
> +    (synopsis
> +     "Emacs interface for sdcv")

Could you explain quickly what SDCV is in the synopsis?

> +    (description
> +     "Translate word by sdcv (console version of Stardict), and display
> +translation use popup tooltip or buffer.")

The description needs to consist of full sentences.

Could you send updated patches?

Regards,
-- 
Nicolas Goaziou