[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#29445] [PATCH] gnu: Add roguebox-adventures.
From: |
Kei Kebreau |
Subject: |
[bug#29445] [PATCH] gnu: Add roguebox-adventures. |
Date: |
Mon, 27 Nov 2017 19:12:54 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Leo Famulari <address@hidden> writes:
> On Sat, Nov 25, 2017 at 04:09:28PM -0500, Kei Kebreau wrote:
>
> Thanks! Overall LGTM, but I have some nitpicks...
>
>> + (package
>> + (name "roguebox-adventures")
>> + (version (string-append "2.1.2." revision "." (string-take commit
>> 7)))
>
> How about (version (git-version "2.1.2" revision commit)) ?
>
>> + (source
>> + (origin
>> + (method git-fetch)
>> + (uri (git-reference
>> + (url
>> "https://git.postactiv.com/themightyglider/RogueBoxAdventures")
>> + (commit commit)))
>
> And similarly, (file-name (git-file-name name version))
>
>> + (replace 'build
>> + (lambda* (#:key outputs #:allow-other-keys)
>> + (let* ((out (assoc-ref outputs "out"))
>> + (data (string-append
> ^
> There are too many spaces here --| and also in the install phase.
>
>> + (mkdir-p bin)
>> + (mkdir-p doc)
>> +
>> + (copy-file "main.py"
>> + (string-append bin "/roguebox-adventures"))
>> + (chmod (string-append bin "/roguebox-adventures") #o555)
>> +
>> + (for-each (lambda (file)
>> + (copy-recursively file
>> + (string-append data "/" file)))
>> + '("AUDIO" "FONT" "GRAPHIC" "LIB" "LICENSE"
>> + "icon_big.png" "icon_small.png"))
>> +
>> + (copy-file "DOC" (string-append doc "/roguebox-adventures"))
>
> And we could simplify a bit by using install-file instead of mkdir-p and
> copy-file.
Is there a way to rename the file when using install-file? It's a bit
weird to have to run "main.py" to launch the game.
signature.asc
Description: PGP signature