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

2016-05-16 Thread Leo Famulari
On Mon, May 16, 2016 at 02:06:51PM -0400, Leo Famulari wrote: > On Mon, May 16, 2016 at 12:45:57PM -0500, Christopher Allan Webber wrote: > > Leo Famulari writes: > > > My opinion is that 3 or 4 GB is not a very small amount of RAM for a > > > personal computer. > > > > > > I think that allowing us

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

2016-05-16 Thread 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 > >> > bui

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

2016-05-16 Thread Leo Famulari
On Mon, May 16, 2016 at 12:45:57PM -0500, Christopher Allan Webber wrote: > Leo Famulari writes: > > 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 takin

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

2016-05-16 Thread Christopher Allan Webber
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 b

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

2016-05-16 Thread 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 cc1pl

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

2016-05-13 Thread Leo Famulari
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

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

2016-05-13 Thread Alex Griffin
I don't know whether parallel builds should be enabled or disabled. My initial thought was that it makes sense if it allows ledger to build on more machines. On the other hand, the more general fix (one that will also work on lots of other packages) is simply to allocate more swap on those machines

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

2016-05-13 Thread Alex Kost
Alex Griffin (2016-05-12 19:31 +0300) wrote: > On Thu, May 12, 2016, at 04:12 AM, Alex Kost wrote: >> Hello, was this package built successfully for you? I tried it but the >> build phase failed (I'm attaching the last part of the build process [1] >> just in case). I don't know, maybe I just do

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

2016-05-12 Thread Leo Famulari
On Thu, May 12, 2016 at 11:31:14AM -0500, Alex Griffin wrote: > On Thu, May 12, 2016, at 04:12 AM, Alex Kost wrote: > > Hello, was this package built successfully for you? I tried it but the > > build phase failed (I'm attaching the last part of the build process [1] > > just in case). I don't kn

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

2016-05-12 Thread Alex Griffin
On Thu, May 12, 2016, at 04:12 AM, Alex Kost wrote: > Hello, was this package built successfully for you? I tried it but the > build phase failed (I'm attaching the last part of the build process [1] > just in case). I don't know, maybe I just don't have enough memory > (3GB) to build it (I had s

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

2016-05-12 Thread Alex Kost
Alex Griffin (2016-05-12 06:28 +0300) wrote: > Here's an updated patch with the suggested changes. I also re-enabled > the shared library and python module, and built html docs, and a few > other things. This turned out to be a much more challenging package than > I expected! Hopefully it is final

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

2016-05-11 Thread Alex Griffin
Here's an updated patch with the suggested changes. I also re-enabled the shared library and python module, and built html docs, and a few other things. This turned out to be a much more challenging package than I expected! Hopefully it is finally finished or very close to finished. Thanks, -- Al

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

2016-05-09 Thread Leo Famulari
On Mon, May 09, 2016 at 10:23:22PM +0200, Ludovic Courtès wrote: > Leo Famulari skribis: > > Good points. My original thought when bringing this up was that we also > > distribute the source code with `guix package --source`. > > Right, but the source carries its own license anyway in ‘COPYING’,

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

2016-05-09 Thread Ludovic Courtès
Leo Famulari skribis: > On Sat, May 07, 2016 at 09:10:48PM -0500, Alex Griffin wrote: >> On Sat, May 7, 2016, at 05:23 PM, Leo Famulari wrote: [...] >> > There are some GPL'd files in 'contrib/', 'lisp/', and 'python/res/' >> > >> > The file 'tools/update_copyright_year' has an Expat license.

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

2016-05-08 Thread Leo Famulari
On Sat, May 07, 2016 at 09:10:48PM -0500, Alex Griffin wrote: > On Sat, May 7, 2016, at 05:23 PM, Leo Famulari wrote: > > What's the story with the library? Are there any programs that use it? > > Not that I know of, which is why I didn't include it (well, also I > couldn't figure out how to tell

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

2016-05-07 Thread Alex Griffin
fin From b97f82dc6bed962fdc59ad674785cb7122846f4a Mon Sep 17 00:00:00 2001 From: Alex Griffin Date: Sat, 7 May 2016 12:20:47 -0500 Subject: [PATCH 2/3] gnu: Add ledger. * gnu/packages/finance.scm (ledger): New variable. --- gnu/packages/finance.scm | 73 +++

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

2016-05-07 Thread Leo Famulari
On Sat, May 07, 2016 at 02:20:16PM -0500, Alex Griffin wrote: > Here's an updated patch which doesn't delete the ledger-mode docs. I had > initially removed them from this package thinking they belonged in > emacs-ledger-mode, then forgot about it before submitting. Turns out > it's more complicate

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

2016-05-07 Thread Alex Griffin
the cmake-build-system. Sorry about that, -- Alex Griffin From cb279e6edc2ed9c3351e4ab9a200d334c6094df0 Mon Sep 17 00:00:00 2001 From: Alex Griffin Date: Sat, 7 May 2016 12:20:47 -0500 Subject: [PATCH 2/3] gnu: Add ledger. * gnu/packages/finance.scm (ledger): New variable. --- gnu/packages/f

[PATCH 2/3] gnu: Add ledger.

2016-05-07 Thread Alex Griffin
Sep 17 00:00:00 2001 From: Alex Griffin Date: Sat, 7 May 2016 12:20:47 -0500 Subject: [PATCH 2/3] gnu: Add ledger. * gnu/packages/finance.scm (ledger): New variable. --- gnu/packages/finance.scm | 61 1 file changed, 61 insertions(+) diff --git a