Subject: Re: [PATCH] emacs-alchemist: add alchemist-server output

From: Liliana Marie Prikler <[email-masked]>
Date: Sun, 06 Oct 2024 15:40:03 +0200

Am Samstag, dem 05.10.2024 um 21:25 +0000 schrieb
[email-masked]:
> From: emma thompson <[email-masked]>
> 
> ---
>  gnu/packages/emacs-xyz.scm | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index 09ac57a464..5567c38a3a 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -4046,6 +4046,15 @@ (define-public emacs-alchemist
>      (build-system emacs-build-system)
>      (propagated-inputs
>       (list emacs-elixir-mode emacs-dash emacs-company emacs-pkg-
> info))
> +    (arguments
Prefer the new style, that is (list #:phases ...)
> +     '(#:phases (modify-phases %standard-phases
> +                  (add-after 'install 'install-server
> +                    (lambda* (#:key outputs #:allow-other-keys)
> +                      (let* ((out (assoc-ref outputs "out"))
> +                             (server (string-append out
> "/share/emacs/site-lisp/alchemist-1.8.2/alchemist-server")))
Is there a better way to encode this path?  You might also want to look
into #:include, which is already provided by emacs-build-system.
> +                        (mkdir-p server)
> +                        (copy-recursively "alchemist-server" server)
> +                        (delete-file-recursively (string-append
> server "/test"))))))))
Cheers