Subject: Re: [bug#37358] Add emacs-doom-themes

From: Ludovic Courtès <[email-masked]>
Date: Wed, 11 Sep 2019 00:03:29 +0200

Hello,

Tim Gesthuizen <[email-masked]> skribis:

>>>From 3fc45dd06f11523f2f7494bcd69790a879635ae4 Mon Sep 17 00:00:00 2001
> From: Tim Gesthuizen <[email-masked]>
> Date: Mon, 9 Sep 2019 18:48:31 +0200
> Subject: [PATCH] gnu: Add emacs-doom-themes
>
> * gnu/packages/emacs-xyz.scm (emacs-doom-themes): New variable.

[...]

> +       (modify-phases %standard-phases
> +         (add-after 'unpack
> +             'move-themes
> +           (lambda _
> +             (copy-recursively "themes/" "."))))))

Why is this needed?

I could suggest:

  (for-each (lambda (file)
              (rename-file file (string-append (dirname file) "/..")))
            (find-files themes))

… which would really move files, but it also more verbose.

WDYT?

Thank you,
Ludo’.