On Sat, Feb 13, 2016 at 09:11:17AM +0100, Jan Nieuwenhuizen wrote: > Nice! New patch set attached.
Very good, I pushed the first two. Concerning the commit message, it should read "gnu: ocaml: ..." in case a package is modified. > 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. Maybe it would also warrant an upstream bug report? Similarly for the make-png patch. Is this an upstream bug? Andreas PS: The game starts and shows the maze, it looks quite nice! 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...