Re: [PATCH] gnu: Add openttd-engine

2016-04-15 Thread Alex Kost
Albin (2016-04-14 12:51 +0300) wrote: > Thanks! Just one little thing: in the TODO, please change > "openttd-opensfx" to "openttd-openmsx". This will give the user free > GPLv2 licensed music (along with the graphics) and should make a good > out-of-the-box experience even without the non-free (

Re: [PATCH] gnu: Add openttd-engine

2016-04-14 Thread Albin
Thanks! Just one little thing: in the TODO, please change "openttd-opensfx" to "openttd-openmsx". This will give the user free GPLv2 licensed music (along with the graphics) and should make a good out-of-the-box experience even without the non-free (NC) opensfx sound effects that I guess we shoul

Re: [PATCH] gnu: Add openttd-engine

2016-04-14 Thread Alex Kost
Albin (2016-04-13 05:09 +0300) wrote: > Here is a new patch for OpenTTD. > > I have > > * changed the name to openttd-engine (so that we can later add game data > as companion packages), I'm going to make this 'openttd-engine' variable private and to add the following lines after it (I'm not sure

Re: [PATCH] gnu: Add openttd-engine

2016-04-14 Thread Alex Kost
Albin (2016-04-13 05:14 +0300) wrote: > I've also changed the download URL to the official one and the package > builds fine. I think this happened because you already had the source in the store, so it wasn't redownloaded from the official url (which would fail). FYI that downloading bug was fi

Re: [PATCH] gnu: Add openttd-engine

2016-04-12 Thread Albin
I've also changed the download URL to the official one and the package builds fine. Den 2016-04-13 kl. 04:09, skrev Albin: > Here is a new patch for OpenTTD. > > I have > > * changed the name to openttd-engine (so that we can later add game data > as companion packages), > * removed the "Unix"->

[PATCH] gnu: Add openttd-engine

2016-04-12 Thread Albin
ssage. Best regards, Albin From 15ca9395ef42c89a0d4d3a3f9f9212e1cb0f1db6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Albin=20S=C3=B6derqvist?= Date: Wed, 13 Apr 2016 03:50:36 +0200 Subject: [PATCH] gnu: Add openttd-engine. * gnu/packages/games.scm (openttd-engine): New variable.

Re: [PATCH] gnu: Add openttd

2016-04-12 Thread Leo Famulari
On Tue, Apr 12, 2016 at 08:26:25PM +0300, Alex Kost wrote: > Here it is (don't forget to uncomment "lzo" input): > >(lambda* (#:key inputs outputs #:allow-other-keys) > (let ((out (assoc-ref outputs "out")) >(lzo (assoc-ref inputs "lzo"))) >

Re: [PATCH] gnu: Add openttd

2016-04-12 Thread Leo Famulari
On Tue, Apr 12, 2016 at 04:55:20PM +0200, Albin wrote: > Thank you Leo for taking the time to inspect and install the package. Thanks for taking the time to work on it :) > Den 2016-04-12 kl. 03:09, skrev Leo Famulari: > > On Tue, Apr 12, 2016 at 12:41:58AM +0200, Albin wrote: > >> +

Re: [PATCH] gnu: Add openttd

2016-04-12 Thread Alex Kost
Albin (2016-04-12 17:55 +0300) wrote: > Thank you Leo for taking the time to inspect and install the package. > > Den 2016-04-12 kl. 03:09, skrev Leo Famulari: >> On Tue, Apr 12, 2016 at 12:41:58AM +0200, Albin wrote: >>> + ;; GNU's not Unix so let's modify the exit dialog. >>> +

Re: [PATCH] gnu: Add openttd

2016-04-12 Thread Albin
Thank you Leo for taking the time to inspect and install the package. Den 2016-04-12 kl. 03:09, skrev Leo Famulari: > On Tue, Apr 12, 2016 at 12:41:58AM +0200, Albin wrote: >> + ;; GNU's not Unix so let's modify the exit dialog. >> + (substitute* (find-files "src/la

Re: [PATCH] gnu: Add openttd

2016-04-12 Thread Albin
Den 2016-04-12 kl. 10:30, skrev Alex Kost: > Leo Famulari (2016-04-12 04:09 +0300) wrote: > >> On Tue, Apr 12, 2016 at 12:41:58AM +0200, Albin wrote: >>> Subject: [PATCH] gnu: Add openttd. >> >> Can you add the "changelog" to the commit message? See comm

Re: [PATCH] gnu: Add openttd

2016-04-12 Thread Alex Kost
Leo Famulari (2016-04-12 04:09 +0300) wrote: > On Tue, Apr 12, 2016 at 12:41:58AM +0200, Albin wrote: >> Subject: [PATCH] gnu: Add openttd. > > Can you add the "changelog" to the commit message? See commit 6dc609985 > for a recent example of the desired format.

Re: [PATCH] gnu: Add openttd

2016-04-11 Thread Leo Famulari
On Tue, Apr 12, 2016 at 12:41:58AM +0200, Albin wrote: > Hi! > > This is my first package for the Guix project. I've done my best to > follow all the guidelines. Thanks for the patch! > Subject: [PATCH] gnu: Add openttd. Can you add the "changelog" to the commit

[PATCH] gnu: Add openttd

2016-04-11 Thread Albin
00:00:00 2001 From: =?UTF-8?q?Albin=20S=C3=B6derqvist?= Date: Tue, 12 Apr 2016 00:13:08 +0200 Subject: [PATCH] gnu: Add openttd. --- gnu/packages/games.scm | 69 ++ 1 file changed, 69 insertions(+) diff --git a/gnu/packages/games.scm b/gnu/pac