From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Nieuwenhuizen Subject: Re: [PATCH] gnu: Add laby. Date: Sat, 13 Feb 2016 15:09:59 +0100 Message-ID: <878u2oaeh4.fsf@gnu.org> References: <87egcleiig.fsf@gnu.org> <87fux1y0vg.fsf@dustycloud.org> <20160211221839.73cf23b5@debian-netbook> <87k2madh1x.fsf@gnu.org> <20160212180905.GA8954@debian> <87pow19gii.fsf@gnu.org> <20160213084329.GA10278@debian> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:56700) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aUatw-0007J7-IB for guix-devel@gnu.org; Sat, 13 Feb 2016 09:10:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aUats-00045H-Bj for guix-devel@gnu.org; Sat, 13 Feb 2016 09:10:20 -0500 In-Reply-To: <20160213084329.GA10278@debian> (Andreas Enge's message of "Sat, 13 Feb 2016 09:43:29 +0100") 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-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Andreas Enge Cc: guix-devel@gnu.org Andreas Enge writes: > Very good, I pushed the first two. Concerning the commit message, it shou= ld > read "gnu: ocaml: ..." in case a package is modified. Ah okay...and Yay, thanks! >> Subject: [PATCH 3/3] gnu: Add laby. >> * gnu/packages/patches/laby-make-install.patch: New file. >> * gnu/packages/patches/laby-make-png.patch: New file. >> * gnu-system.am (dist_patch_DATA): Add them. >> * gnu/packages/games.scm (laby): New variable. > > Here I would not use a patch. The policy is not totally clear (and not > written up), but if an additional phase for installation is needed, I > would do it in a custom phase on the scheme side instead of patching the > makefile. See, for instance, the tinyxml package in xml.scm. Yes, I see. Good to know, thanks. > Maybe it would also warrant an upstream bug report? Yes, I have sent the patches upstream. > Similarly for the make-png patch. Is this an upstream bug? I think it's a problem with Guix's ocaml or gtk not rendering svg images; so I wanted to first `see if png works' and then figure out what's going on...So this could be temporary, depending on the availability of svg rendering, not sure what to do here. So...please advise. Greetings, Jan > PS: The game starts and shows the maze, it looks quite nice! :-) Laby looks for programming languages installed and offer only those that are available. So while it does not depend on guile, gcc, ocaml etc..., those packages make it more functional. > PPS: You do not happen to use unison? I think your previous patches would > allow to build it with gtk support, and unlike me, you seem to know > what you are doing... No, I have used SparkleShare until it became obvious there won't be a headless version and I reverted to plain git. I'll have a look. Does it use git for a backend? --=20 Jan Nieuwenhuizen | GNU LilyPond http://lilypond.org Freelance IT http://JoyofSource.com | Avatar=C2=AE http://AvatarAcademy.nl= =20=20