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

[Ricardo Wurmus]

Alex Kost <address@hidden> writes:

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

Okay.

>> +                           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" -------^

Thanks for the reminder.

> Otherwise, LGTM.

Thank you for the review.  I pushed it to master.

~~ Ricardo