Re: [PATCH] gnu: Add laby.

[Andreas Enge <andreas@enge.fr>]

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...