guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#26160: [PATCH] gnu: Add deutex.


From: Leo Famulari
Subject: bug#26160: [PATCH] gnu: Add deutex.
Date: Sun, 19 Mar 2017 18:39:22 -0400
User-agent: Mutt/1.8.0 (2017-02-23)

On Sat, Mar 18, 2017 at 04:24:48PM -0400, Kei Kebreau wrote:
> gnu/packages/game-development.scm (deutex): New variable.

> +        ;; "make install" is broken for this package.

It's helpful to say how it's broken. That will make it easier to know
if `make install` is fixed in future versions of DeuTex.

> +        (replace 'install
> +                 (lambda* (#:key outputs #:allow-other-keys)
> +                   (let* ((out (assoc-ref %outputs "out"))
> +                          (bin (string-append out "/bin"))
> +                          (share (string-append out "/share")))
> +                     (mkdir-p bin)

This procedure is redundant; (install-file) will create the directory.

Otherwise, LGTM.

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]