From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albin Subject: Re: [PATCH] gnu: Add openttd Date: Tue, 12 Apr 2016 16:55:20 +0200 Message-ID: <570D0C58.5050905@fripost.org> References: <570C2836.7030100@fripost.org> <20160412010929.GA22784@jasmine> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:37116) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1apziy-0005ky-6J for guix-devel@gnu.org; Tue, 12 Apr 2016 10:55:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1apziu-0002Gb-3i for guix-devel@gnu.org; Tue, 12 Apr 2016 10:55:28 -0400 Received: from giraff.fripost.org ([178.16.208.44]:54119 helo=outgoing.fripost.org) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1apzit-0002GP-MW for guix-devel@gnu.org; Tue, 12 Apr 2016 10:55:24 -0400 In-Reply-To: <20160412010929.GA22784@jasmine> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Leo Famulari Cc: guix-devel@gnu.org 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/lang/" "\\.txt") >> + ((":Unix") ":GNU")))))) >=20 > I don't think this is necessary. Maybe not. I just wanted to exercise my freedom a little. The above snippet lets you see this dialog box: "Are you sure you want to exit OpenTTD and return to GNU" ...instead of "...Unix", which I thought would make the Guix users happy. It works for 54 language versions, but not Arabic, Esperanto, Hebrew and Urdu where "Unix" is called something else. I'm not going to insist on keeping it, but remember that the "Words to avoid" article (https://www.gnu.org/philosophy/words-to-avoid.html) states: "To call the whole system 'Linux' is both unfair and confusing." Isn't it likewise wrong to call "GNU" "Unix"? >=20 >> + (lambda* (#:key outputs #:allow-other-keys) >> + ;; The build scripts don't understand the option >> + ;; `--enable-fast-install'. >> + (let ((out (assoc-ref outputs "out"))) >> + (zero? >> + (system* "./configure" >> + ;; At the moment, `lzo' must be disabled >> + ;; because the `config.lib' script cannot >> + ;; find it. It is only necessary for loadin= g >> + ;; saved games generated by early versions o= f >> + ;; OpenTTD. >> + (string-append "--prefix=3D" out) >> + "--without-liblzo2" >> + ;; Put the binary in `bin' instead of `games= ' >> + "--binary-dir=3Dbin")))))))) >=20 > I see this option in `./configure --help`: >=20 > --with-liblzo2[=3Dliblzo2.a] enables liblzo2 support Nice! > Does it work to provide the path to that option? Our lzo package does > build a static library, if that is what OpenTTD requires. Yes, I could make it work by first installing 'lzo' and then add `--with-liblzo2=3D/gnu/store/[...]/liblzo2.a` to the package definition. What is the proper way to do this? >=20 >> + (description "OpenTTD is a game in which you transport goods and >> +passengers by land, water and air. It is a free implementation of >> +Transport Tycoon Deluxe but with many enhancements including >> +multiplayer mode, internationalization support, conditional orders an= d >> +the ability to clone, autoreplace and autoupdate vehicles.") >=20 > All software in Guix is free software, so I don't think it's necessary > to call this a "free implementation". How about "re-implementation"? OK. >> + ;; The software contains an in-game downloader from which the use= r >> + ;; may find non-functional data licensed under different terms. >=20 > I don't know the significance of this. Hopefully somebody else will > comment on this subject. >=20 > Upon starting the program, I was prompted to download something related > to graphics. I agreed, and the game seemed to download OpenGFX [0] and > launch correctly, although it also warned me about needed to download > some sound files. Yes, it's not ideal that you are immediately prompted to download and install something like this. As you said, the package that gets installed when you agree is OpenGFX (8 MiB). It has the same license as the game itself, i.e. GPLv2, just like the other available graphics sets "NightGFX" (3.4 MiB) and "zBase" (273 MiB). The music files that you can download are: * Modern Motion (music): CC-BY-NC-ND 3.0 * OpenMSX: GPLv2 * Scott Joplin Anthology: public domain * Traditional Winter Holiday Music: public domain For sound effects only one package is currently available: * OpenSFX: CC Sampling Plus 1.0 I can understand that packages that restrict commercial redistribution may not be included in Guix, but the FSDG doesn't discuss the particular case of packages that *recommend* non-free, non-functional data where only non-commercial redistribution is permitted. How does other free distros deal with this? The Arch project distributes openttd, openttd-opengfx and openttd-opensfx as separate packages (where the package manager recommends the latter two upon installation of openttd). Parabola does the same, but removes openttd-opensfx from their repository. You can still download the sound effects afterwards however. Do you think we should follow Parabola in this case or take a different approach? >=20 >> + (license license:gpl2))) >=20 > Section 10.0 of 'readme.txt' describes the licensing of 3rd party > components. If any of those are included, be sure to add them to a list > here. You are right! Thanks. >=20 > [0] > http://dev.openttdcoop.org/projects/opengfx >=20 Best regards, Albin