Re: [PATCH] gnu: Add laby.

2016-08-11 Thread 宋文武
Jan Nieuwenhuizen writes: > [...] > > Gtk now has librsvg working, so Laby works without the ugly png patch > hack. > > From dcf2cde075cf59481ebfc98db18c6ca709bf5519 Mon Sep 17 00:00:00 2001 > From: Jan Nieuwenhuizen > Date: Sun, 7 Feb 2016 12:57:40 +0100 > Subject: [

Re: [PATCH] gnu: Add laby.

2016-08-10 Thread Jan Nieuwenhuizen
Mon Sep 17 00:00:00 2001 From: Jan Nieuwenhuizen Date: Sun, 7 Feb 2016 12:57:40 +0100 Subject: [PATCH] gnu: Add laby. * gnu/packages/patches/laby-make-install.patch: New file. * gnu/local.mk (dist_patch_DATA): gnu-system.am: Add it. * gnu/packages/games.scm (laby): New variable. --- gnu/local.mk

Re: [PATCH] gnu: Add laby.

2016-03-06 Thread Jan Nieuwenhuizen
eb 2016 12:57:40 +0100 Subject: [PATCH] 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. --- gnu-system.am

Re: [PATCH] gnu: Add laby.

2016-03-06 Thread Andreas Enge
Hi Jan, I am afraid that my digression on unison has made us forget laby. If I remember well, you had a package more or less ready? Could you maybe send the patch again? Andreas

Re: [PATCH] gnu: Add laby.

2016-02-13 Thread Jan Nieuwenhuizen
Andreas Enge writes: > Since there is no terrible hurry, I would suggest to examine the svg rendering > issue first; if you do not find a solution in reasonable time, we can still > push the png patch. Sure. > No, it just synchronises and does not allow to roll back. On the other hand, > it also

Re: [PATCH] gnu: Add laby.

2016-02-13 Thread Andreas Enge
On Sat, Feb 13, 2016 at 03:09:59PM +0100, Jan Nieuwenhuizen wrote: > 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 renderi

Re: [PATCH] gnu: Add laby.

2016-02-13 Thread Jan Nieuwenhuizen
Andreas Enge writes: > Very good, I pushed the first two. Concerning the commit message, it should > 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/packa

Re: [PATCH] gnu: Add laby.

2016-02-13 Thread Jan Nieuwenhuizen
Andreas Enge writes: > "git am" complained about whitespace issues with tabs, but I suppose these > are okay in make files :-) :-) >> +(inputs >> + `(("camlp4" ,camlp4) >> + ("m4" ,m4) >> + ("ocaml" ,ocaml))) > > I think all of these should be native-inputs; the first two are

Re: [PATCH] gnu: Add laby.

2016-02-13 Thread Andreas Enge
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

Re: [PATCH] gnu: Add laby.

2016-02-13 Thread Andreas Enge
On Fri, Feb 12, 2016 at 09:43:47PM +0100, Jan Nieuwenhuizen wrote: > Ah yes it is inpired by the Nix package, it is store-specific. Ocaml > wants its "core" libraries all in the same prefix, store won't allow > that. Okay. > When I hit this, I wondered if we have a way to import/convert packages

Re: [PATCH] gnu: Add laby.

2016-02-13 Thread Jan Nieuwenhuizen
Andreas Enge writes: >> + (native-search-paths >> +(list (search-path-specification >> + (variable "OCAMLPATH") >> + (files (list (string-append "lib/ocaml")) > > I have doubts about this search path. Normally, a search path is registered > with the program that needs

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Jan Nieuwenhuizen
Andreas Enge writes: > At the top of the patch file, could you add a quick explanation where it comes > from and what it is needed for? Is it Guix specific, or an upstream > bug? Ah yes it is inpired by the Nix package, it is store-specific. Ocaml wants its "core" libraries all in the same prefi

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Jan Nieuwenhuizen
Andreas Enge writes: >> + (system* >> +"./configure" >> +"-bindir" (string-append out "/bin") >> +"-config" (string-append out "/etc/ocamfind.conf") >> +"-mandir" (string-append ou

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Andreas Enge
Hello, On Thu, Feb 11, 2016 at 11:21:30PM +0100, Jan Nieuwenhuizen wrote: > Ah... And also, ocaml-findlib was missing and lablgtk did not compile > cmxa files, which laby needs. So, three patches to get a running Laby. > + (native-search-paths > +(list (search-path-specification > +

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Andreas Enge
On Thu, Feb 11, 2016 at 11:21:30PM +0100, Jan Nieuwenhuizen wrote: > + #:phases (modify-phases %standard-phases > + (replace > + 'configure > + (lambda* (#:key inputs outputs #:allow-other-keys) > + (let ((out (assoc-ref

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Andreas Enge
Hello again, I notice that this: > Subject: [PATCH 2/3] gnu: Add ocaml-findlib. is independent, so I also had a look. In fact, it contains the copyright line, so that is already settled! At the top of the patch file, could you add a quick explanation where it comes from and what it is needed for

Re: [PATCH] gnu: Add laby.

2016-02-11 Thread Jan Nieuwenhuizen
Efraim Flashner writes: > The patch isn't registered in gnu-system.am, so that should be why. Ah... And also, ocaml-findlib was missing and lablgtk did not compile cmxa files, which laby needs. So, three patches to get a running Laby. > That's all I have to add, other than I'm also looking forw

Re: [PATCH] gnu: Add laby.

2016-02-11 Thread Efraim Flashner
On Tue, 09 Feb 2016 14:28:35 -0800 Christopher Allan Webber wrote: > Jan Nieuwenhuizen writes: > > [...] > > Now this looks like a lot of fun! > > Some of these lines are over 80 characters though.. could you fix that? > > I tried running this though and the patch it provided didn't seem t

Re: [PATCH] gnu: Add laby.

2016-02-09 Thread Christopher Allan Webber
Jan Nieuwenhuizen writes: > Hi, > > And now a real fun package...If you don't know Laby, maybe you want to > have a look at it. It's a programming game that my daughters happen to > like very much. They started programming with this without knowing they > did; they were simply playing a game. >

[PATCH] gnu: Add laby.

2016-02-09 Thread Jan Nieuwenhuizen
ed it. So, her first programming was done in Guile Scheme; apparently the easiest syntax of all, if you learn young. Greetings, Jan >From fcd853a5c9b13019409d69db48f6bac0023b7b30 Mon Sep 17 00:00:00 2001 From: Jan Nieuwenhuizen Date: Sun, 7 Feb 2016 12:57:40 +0100 Subject: [PATCH] gnu: Add lab