Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Ludovic Courtès
Alex Kost skribis: > Ouch, yes, 'second' is 'cadr', my bad. Actually I think 'first', > 'second', etc. are not much better then 'car', 'cadr', etc. It is > usually possible and desirable to avoid such things, although I confess > I use 'car'-s sometimes (when Ludovic doesn't see :-)). Bah, cau

Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Ludovic Courtès
Hi! Alex Vong skribis: > On 06/10/2015, Ludovic Courtès wrote: [...] >> It’s not uncommon, indeed. However, the details on how to bootstrap are >> not standard: Often ‘autoreconf -vfi’ will do, sometimes it’s >> ‘./bootstrap’, sometimes ‘./autogen.sh’, etc. >> >> Now the proposed build syste

Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Alex Kost
Mathieu Lirzin (2015-10-06 18:06 +0300) wrote: > Alex Vong writes: > >> On 05/10/2015, Mathieu Lirzin wrote: >>> Alex Vong writes: + +;;; Procedures to manupulate build flags, similar to dpkg-buildflags. +;;; +;;; The data strcture flag-list is constrcuted by (flag-list

Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Mathieu Lirzin
Alex Vong writes: > On 05/10/2015, Mathieu Lirzin wrote: >> Alex Vong writes: >>> + >>> +;;; Procedures to manupulate build flags, similar to dpkg-buildflags. >>> +;;; >>> +;;; The data strcture flag-list is constrcuted by (flag-list >>> ...) >> ^^ >>

Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Alex Kost
Alex Vong (2015-10-06 16:43 +0300) wrote: > Hi Alex, > > On 06/10/2015, Alex Kost wrote: [...] >>> +;;; selectors of flag-sublist >>> +(define (flag-type flag-sublist) >>> + (car flag-sublist)) >>> +(define (flag-string-list flag-sublist) >>> + (cdr flag-sublist)) >> >> IMO it is clearer to wri

Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Alex Vong
Hi Ludovic, On 06/10/2015, Ludovic Courtès wrote: > Alex Vong skribis: > >> On 05/10/2015, Mathieu Lirzin wrote: >>> Alex Vong writes: > > [...] > >>> I'm not competent enough to judge if it's a useful build-system to add >>> but >>> this should be done in another commit and in >>> “guix/guix/

Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Alex Vong
Hi Alex, On 06/10/2015, Alex Kost wrote: > Hello and thank you for contributing. This is a tremendous patch for > the first attempt! > > As Mathieu noted, if this auxiliary code for adding 'flags' is needed, > it should be separated from the package commit and it shouldn't be > placed in the pac

Re: [PATCH] gnu: Add Mlucas.

2015-10-06 Thread Ludovic Courtès
Alex Vong skribis: > On 05/10/2015, Mathieu Lirzin wrote: >> Alex Vong writes: [...] >> I'm not competent enough to judge if it's a useful build-system to add but >> this should be done in another commit and in >> “guix/guix/build/bootstap-build-system.scm” if yes. >> > I should be more a new

Re: [PATCH] gnu: Add Mlucas.

2015-10-05 Thread Alex Vong
Hi Mathieu, On 05/10/2015, Mathieu Lirzin wrote: > Alex Vong writes: > >> From e5155b52f636bfee849268b19b81f5b6608540fd Mon Sep 17 00:00:00 2001 >> From: Alex Vong >> Date: Mon, 5 Oct 2015 12:49:49 +0800 >> Subject: [PATCH] gnu: Add Mlucas. >> >> * g

Re: [PATCH] gnu: Add Mlucas.

2015-10-05 Thread Alex Kost
Hello and thank you for contributing. This is a tremendous patch for the first attempt! As Mathieu noted, if this auxiliary code for adding 'flags' is needed, it should be separated from the package commit and it shouldn't be placed in the package module. You will probably receive useful comment

Re: [PATCH] gnu: Add Mlucas.

2015-10-05 Thread Mathieu Lirzin
Alex Vong writes: > From e5155b52f636bfee849268b19b81f5b6608540fd Mon Sep 17 00:00:00 2001 > From: Alex Vong > Date: Mon, 5 Oct 2015 12:49:49 +0800 > Subject: [PATCH] gnu: Add Mlucas. > > * gnu/packages/mlucas.scm: New file. > * gnu-system.am (GNU_SYSTEM_MODULES): Regist

[PATCH] gnu: Add Mlucas.

2015-10-05 Thread Alex Vong
From e5155b52f636bfee849268b19b81f5b6608540fd Mon Sep 17 00:00:00 2001 From: Alex Vong Date: Mon, 5 Oct 2015 12:49:49 +0800 Subject: [PATCH] gnu: Add Mlucas. * gnu/packages/mlucas.scm: New file. * gnu-system.am (GNU_SYSTEM_MODULES): Register it. --- gnu-system.am | 1 + gnu/packages