Re: [PATCH] gnu: Add emacs-multiple-cursors.

[Alex Kost]

Ricardo Wurmus (2016-06-17 23:24 +0300) wrote:

> * gnu/packages/emacs.scm (emacs-multiple-cursors): New variable.
> ---
>  gnu/packages/emacs.scm | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index bbb4baa..62e14cb 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -1356,6 +1356,27 @@ front-ends.  Company comes with many back-ends, e.g. 
> @code{company-elisp}.
>  These are distributed in separate files and can be used individually.")
>      (license license:gpl3+)))
>  
> +(define-public emacs-multiple-cursors
> +  (package
> +    (name "emacs-multiple-cursors")
> +    (version "1.4.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append 
> "https://github.com/magnars/multiple-cursors.el/archive/";

I would make this line a bit shorter.

> +                           version ".tar.gz"))
> +       (file-name (string-append name "-" version ".tar.gz"))
> +       (sha256
> +        (base32
> +         "0hihihlvcvzayg5fnqzcg45fhvlmq6xlq58syy00rjwbry9w389k"))))
> +    (build-system emacs-build-system)
> +    (home-page "https://github.com/magnars/multiple-cursors.el";)
> +    (synopsis "Multiple cursors for Emacs.")

Oops, don't forget about "guix lint" -------^

Otherwise, LGTM.

> +    (description
> +     "This package adds support to Emacs for editing text with multiple
> +simultaneous cursors.")
> +    (license license:gpl3+)))
> +
>  (define-public typo
>    (package
>      (name "emacs-typo")

-- 
Alex