Re: [PATCH 2/3] gnu: Add ledger.

[Alex Kost]

Leo Famulari (2016-05-14 05:49 +0300) wrote:

> On Fri, May 13, 2016 at 10:16:29PM +0300, Alex Kost wrote:
>> Alex Griffin (2016-05-12 19:31 +0300) wrote:
>> > Yes, it builds fine for me. It looks like the important line in your
>> > build log is "c++: internal compiler error: Killed (program cc1plus)",
>> > which could be from running out of memory. Does it still happen if you
>> > add `#:parallel-build? #f` to the build system arguments?
>> 
>> Oh indeed, if parallel-build is disabled, it is built successfully.
>> Thank you (and Leo)!
>> 
>> So I don't know, should ‘#:parallel-build? #f’ be used in the final
>> package recipe?  I guess not, as it looks like a problem on my side (not
>> enough memory for parallel-build).
>
> My opinion is that 3 or 4 GB is not a very small amount of RAM for a
> personal computer.
>
> I think that allowing users with "only" 4 GB RAM to build our ledger
> package is worth it taking twice as long for the rest of us.
>
> Or, users with ≤ 4 GB RAM could make a private variant of ledger that
> disables parallel building. We sometimes suggest that users with
> esoteric requirements or restrictions do something like this, but I
> don't think this is one of those cases.

I have commited this patch without ‘#:parallel-build? #f’.  Let's see if
hydra can build it or not :-)

-- 
Alex