GNU bug report logs - #26160
[PATCH] gnu: Add deutex.

Previous Next

Full log

Message #13 received at 26160-done <at> debbugs.gnu.org (full text, mbox):

From: Kei Kebreau <kei <at> openmailbox.org>
To: Leo Famulari <leo <at> famulari.name>
Cc: 26160-done <at> debbugs.gnu.org
Subject: Re: bug#26160: [PATCH] gnu: Add deutex.
Date: Sun, 19 Mar 2017 21:48:40 -0400

[Message part 1 (text/plain, inline)]

Leo Famulari <leo <at> famulari.name> writes:

> 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.
>

A copy of the updated explanation:

"make install" is broken for this package.
Notably, the binaries overrwrite one another upon installation as
they are all installed to the "bin" file in the output directory,
and the manual page fails to install because the directory for the
manual page is not created.

>> +        (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.
>

Redundant procedure removed.

> Otherwise, LGTM.

Pushed to master as 1a0c4437abbe3a1dd9622cc8a7926b9add74bf79. Thanks for
the review.

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.