Re: [Patch] Tmux Themepack

[Alex Kost]

Leo Famulari (2016-07-10 19:19 +0300) wrote:

> Thanks! I found myself wondering about some of the new module imports,
> in particular (guix build utils), (guix utils), and (gnu packages base).
>
> The latter two are not needed; tmux.scm compiles and tmux and
> tmux-themepack can both be built without them.
>
> I think that (guix build utils) should be required to provide
> copy-recursively, but everything still works even when I remove the
> module. Does anyone know why that is? Is it coming from (guix packages)?

There is no point to use such modules in a package module.
'copy-recursively' is needed only at the build time.  Don't forget that
the value of 'arguments' field is a *quoted* expression.  You can write
anything there, and the package module will still be compiled
successfully (it could fail only during the build).

(guix build utils) also shouldn't be explicitly specified inside
'arguments', as it is already used by gnu-build-system - see
%gnu-build-system-modules in (guix build-system gnu).

-- 
Alex