[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 6/6] gnu: Add emacs-slime.
From: |
Federico Beffa |
Subject: |
Re: [PATCH 6/6] gnu: Add emacs-slime. |
Date: |
Thu, 31 Dec 2015 11:32:35 +0100 |
On Thu, Dec 31, 2015 at 11:17 AM, Alex Kost <address@hidden> wrote:
> Federico Beffa (2015-12-30 20:33 +0300) wrote:
>
> [...]
>> +(define-public emacs-slime
>> + (package
>> + (name "emacs-slime")
>> + (version "2.15")
>> + (source
>> + (origin
>> + (method url-fetch)
>> + (uri (string-append
>> + "https://github.com/slime/slime/archive/v"
>> + version ".tar.gz"))
>> + (sha256
>> + (base32
>> + "0l2z6l2xm78mhh0nczkrmzh2ddb1n911ij9xb6q40zwvx4f8blds"))))
>> + (build-system emacs-build-system)
>> + (inputs
>> + `(("texlive" ,texlive)
>> + ("texinfo" ,texinfo)))
>
> As these inputs are used just for generating documentation, I think they
> should be in 'native-inputs'.
yes. will change that.
>
>> + (arguments
>> + `(#:phases
>> + (modify-phases %standard-phases
>> + (add-before 'install 'configure
>> + (lambda* _
>> + (substitute* "slime.el"
>> + (("inferior-lisp-program \"lisp\"")
>> + "inferior-lisp-program \"sbcl\""))
>> + #t))
>
> Why do you name this phase 'configure'? Actually I think this phase is
> not needed, because 'inferior-lisp-program' should be set by a user to
> whatever implementation (s)he prefers.
It's called configure because I'm configuring a default CL implementation.
The default value in slime is 'lisp'. I only used CLISP and SBCL and
none of them provides a binary called 'lisp'. For this reason I tough
it to be a good idea to provide a default name corresponding to an
existing binary.
Of course the user has to adapt to its own needs.
Thanks.
Fede