GNU bug report logs - #25695
angband [new package]

Previous Next

Package: guix-patches;

Reported by: Christopher Allan Webber <cwebber <at> dustycloud.org>

Date: Sun, 12 Feb 2017 14:02:02 UTC

Severity: normal

Tags: patch

Done: Christopher Allan Webber <cwebber <at> dustycloud.org>

Bug is archived. No further changes may be made.

Full log


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

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Christopher Allan Webber <cwebber <at> dustycloud.org>
Cc: 25695 <at> debbugs.gnu.org
Subject: Re: bug#25695: angband [new package]
Date: Sun, 12 Feb 2017 19:42:52 +0100
Christopher Allan Webber <cwebber <at> dustycloud.org> writes:

>>> +       #:phases (modify-phases %standard-phases
>>> +                  (add-after
>>> +                   'unpack 'autogen.sh
>>
>> Nitpick: move the phase names onto the same line as “add-after”.
>
> Hm, okay, I can do that, but I didn't do it for a reason... emacs
> an indentation style reason!
[…]
>
> Maybe we should put something in Guix's .dir-locals?
>

Do you have “guix-devel-mode” enabled?  It indents the expression just
fine in my Emacs (though I’d add a line break after “#:phases”).

>> “copying.txt” says:
>>
>>  * Adam Bolt's (16x16) graphics may be redistributed and used for any purpose,
>>    with or without modification.
>>
>>  * David Gervais' (32x32) graphics may be redistributed, modified, and used
>>    only under the terms of the Creative Commons Attribution 3.0 licence:
>>    http://creativecommons.org/licenses/by/3.0/
>>
>>  * Shockbolt's (64x64) graphics are distributed under the licence at the end
>>    of this file.  Currently some of the tiles in the 64x64.png tilesheet were
>>    resized from tiles made by David Gervais for the 32x32 set.
>>
>>  * The sounds are licenced under the Creative Commons Attribution-NonCommercial-
>>    Sharealike licence.  They were created by Dubtrain <angband <at> dubtrain.com>.
>>    You can find them in Wave format at http://www.dubtrain.com/angband/.
>
> Hmm, that last one is a problem because I think it doesn't meet our
> licensing requirements.  It's a bit funny because we aren't using the
> graphics or sounds anyway since this is a console-only version of this
> package.  So!  Easy solution: I added a phase that just deleted them all
> from the output.  Problem solved!  (Until someone adds a graphical
> variant.  Then they can deal with it ;))

Could you remove the sounds in a snippet instead?  This ensures that we
don’t redistribute them as part of the source tarball either.

I think it’s okay to push with that change.  Thanks!

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net





This bug report was last modified 8 years and 104 days ago.

Previous Next


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