Re: [PATCH] gnu: Add emacs-clojure-mode

[Alex Kost <alezost@gmail.com>]

Catonano (2016-05-19 13:20 +0300) wrote:

> 2016-05-19 10:54 GMT+02:00 Alex Kost <alezost@gmail.com>:
>
>     > Subject: gnu: Add emacs-clojure-mode
>                                           ^
>     This is a nitpick of course, but we put a period here...
>
>     > * gnu/packages/emacs.scm (emacs-clojure-mode): New variable
>                                                                  ^
>     ... and here.
>
>
> Do you mean like this ?
>
> Subject: gnu: Add emacs-clojure-mode.
>
> * gnu/packages/emacs.scm (emacs-clojure-mode): New variable.

Yes, thanks.

[...]
> So, I edited the patch and attached it again.
> Would you see if it fits ?

Yes, the only thing is: I had to do some manual interventions as this
patch wasn't applied cleanly because it was based on some old commit
(before your smartparens package).

Please, next time make sure that your patch can be applied on the
current master.  And thanks for contributing!

> From 6dc8470baf3493a93da3e3358136e5c9860d4e3a Mon Sep 17 00:00:00 2001
> From: humanitiesNerd <catonano@gmail.com>
> Date: Wed, 18 May 2016 15:41:47 +0200
> Subject: gnu: Add emacs-clojure-mode.
>
> * gnu/packages/emacs.scm (emacs-clojure-mode): New variable.
> ---
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index 32ed722..fb07bc7 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -1612,3 +1612,28 @@ It is built on top of the custom theme support in Emacs 24 or later.")
>      (license license:gpl3+)))
>
>
> +(define-public emacs-clojure-mode
> +  (package
> +    (name "emacs-clojure-mode")
> +    (version "5.3.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "https://github.com/clojure-emacs/clojure-mode/archive/"
> +             version
> +             ".tar.gz"))
> +       (file-name (string-append name "-" version ".tar.gz"))
> +       (sha256
> +         (base32
> +         "0gi8ra3ap5m3mz4qh1yxp2cldn7z9xcxvypznr6rrlc6a9l8s5a6"))))
> +    (build-system emacs-build-system)
> +    (home-page "http://github.com/clojure-emacs/clojure-mode")
> +    (synopsis "Major mode for Clojure code")
> +    (description
> +     "This Emacs package provides font-lock, indentation,3
I removed this extra "3" -----------------------------------^

> +navigation and basic refactoring for the
> +@uref{http://clojure.org, Clojure programming language}
> +
> +Using clojure-mode with paredit or smartparens is highly recommended.")

I removed the word "highly" as I think it's too much :-)
> +    (license license:gpl3+)))
>

Applied as commit 8eeb301¹, thanks!

¹ http://git.savannah.gnu.org/cgit/guix.git/commit/?id=8eeb301de2b4d3086f8e88f8a48bba0e83cf79bc

-- 
Alex