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

[Efraim Flashner]

On Mon, May 16, 2016 at 12:45:57PM -0500, Christopher Allan Webber wrote:
> Leo Famulari writes:
> 
> > 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.
> >
> > Thoughts?
> 
> I guess I'm of a different view... it was only very recently that I
> upgraded to a machine that had > 4GB ram.  And not too long ago I had a
> machine with 2GB of ram for a long time.
> 
> Which, yes maybe that's living in the past... I dunno :)
> 

I wish I could remember, but we do have another build that has a test
disabled because it uses ~4GB of memory.

-- 
Efraim Flashner   <address@hidden>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

signature.asc
Description: PGP signature