From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:44605) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hb9Q2-0001D3-LI for guix-patches@gnu.org; Wed, 12 Jun 2019 16:00:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hb9Fy-0007LZ-Ic for guix-patches@gnu.org; Wed, 12 Jun 2019 15:50:03 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:48787) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hb9Fy-0007LQ-FD for guix-patches@gnu.org; Wed, 12 Jun 2019 15:50:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1hb9Fy-0005oV-CL for guix-patches@gnu.org; Wed, 12 Jun 2019 15:50:02 -0400 Subject: [bug#36182] [PATCH] add freeorion Resent-Message-ID: From: Nicolas Goaziou References: <20190612102219.6cc0d25f@gmail.com> Date: Wed, 12 Jun 2019 21:48:52 +0200 In-Reply-To: <20190612102219.6cc0d25f@gmail.com> (Jesse Gibbons's message of "Wed, 12 Jun 2019 10:22:19 -0600") Message-ID: <871rzyshy3.fsf@nicolasgoaziou.fr> MIME-Version: 1.0 Content-Type: text/plain List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Jesse Gibbons Cc: 36182@debbugs.gnu.org Hello, Jesse Gibbons writes: > Patch is attached. -Jesse Thank you! Some comments follow. > From cca52f508e84ac34e60a3c5167554d7ad2ad6564 Mon Sep 17 00:00:00 2001 > From: Jesse Gibbons > Date: Wed, 12 Jun 2019 10:07:32 -0600 > Subject: [PATCH] add freeorion The commit message is incomplete. It should also include * gnu/packages/games.scm (freeorion): New variable. If you use Emacs, there is a template to automatically fill this. > + (origin > + (method git-fetch) > + (uri (git-reference > + (url "https://github.com/freeorion/freeorion.git") > + (commit "470d0711537804df3c2ca25532f674ab4bec58af"))) Why do you need to use the latest commit instead of the latest stable release? Unless there is a good reason, Guix prefers using stable releases: (commit version) > + (sha256 > + (base32 > + "1wsw632l1cj17px6i88nqjzs0dngp5rsr67n6qkkjlfjfxi69j0f")))) > + (arguments > + '(#:tests? #f)) You should include a comment explaining why tests are removed. It could be, for example, '(#:tests? #f)) ;no test > + (home-page > + "https://www.freeorion.org/index.php/Main_Page") I think "https://www.freeorion.org" is enough, since it points to the page above. > + (description > + "FreeOrion is a free, open source, turn-based space empire and galactic > +conquest (4X) computer game being designed and built by the FreeOrion project. > +FreeOrion is inspired by the tradition of the Master of Orion games, but is not > +a clone or remake of that series or any other game.") It may be useful to explain what the "4X" means, or do not include it at all. Note that in the wiki, "4X" is a link, so you can get further information. > + (license (list license:gpl2 license:cc-by-sa3.0)))) Could you explain, in a comment, why there are two licenses? I assume the former is the project, and the latter is for assets. Would you mind sending an updated patch? Regards, -- Nicolas Goaziou