From: Leo Famulari <leo@famulari.name>
To: Alex Griffin <a@ajgrf.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 2/3] gnu: Add ledger.
Date: Sun, 8 May 2016 23:39:06 -0400 [thread overview]
Message-ID: <20160509033906.GC25977@jasmine> (raw)
In-Reply-To: <1462673448.1215366.601202601.6F8B62D7@webmail.messagingengine.com>
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 cmake that I don't want it to put stuff
> in "$out/lib64"). And Python users will probably prefer beancount to the
> ledger module. (Beancount is a reimplementation of ledger in Python.)
Okay, I think it deserves a "TODO" comment where it is disabled.
> > There are some GPL'd files in 'contrib/', 'lisp/', and 'python/res/'
> >
> > The file 'tools/update_copyright_year' has an Expat license.
> >
> > And, I think that unless we delete the bundled utfcpp, we are
> > distributing it through `guix build --source ledger`, so we should
> > mention its Boost license.
>
> Is the license field for the source tarball or the package that actually
> gets installed?
Good question. Can anyone answer it?
> If it's the former, I have to say that's really
> unintuitive. I was aware of these extra licenses, but none of that code
> gets installed after you build the package. Note that emacs-ledger-mode
> uses the same source tarball and I specified its license as GPL2+.
> Instead of changing the license field in this patch, I added a note
> about the other files to make it clear that changing the build options
> may require the license field to change too. I can change it again if
> that's wrong, but it seems to me that the license field should really be
> about what gets installed.
Good points. My original thought when bringing this up was that we also
distribute the source code with `guix package --source`. I don't think
we have an idiomatic way to specify a different set of licenses for the
source code and the built package.
Advice requested!
next prev parent reply other threads:[~2016-05-09 3:39 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-07 18:38 [PATCH 2/3] gnu: Add ledger Alex Griffin
2016-05-07 19:20 ` Alex Griffin
2016-05-07 22:23 ` Leo Famulari
2016-05-08 2:10 ` Alex Griffin
2016-05-09 3:39 ` Leo Famulari [this message]
2016-05-09 20:23 ` Ludovic Courtès
2016-05-09 21:01 ` Leo Famulari
2016-05-12 3:28 ` Alex Griffin
2016-05-12 9:12 ` Alex Kost
2016-05-12 16:31 ` Alex Griffin
2016-05-12 19:03 ` Leo Famulari
2016-05-13 19:16 ` Alex Kost
2016-05-13 21:05 ` Alex Griffin
2016-05-14 2:49 ` Leo Famulari
2016-05-16 15:03 ` Alex Kost
2016-05-16 17:45 ` Christopher Allan Webber
2016-05-16 18:05 ` Efraim Flashner
2016-05-16 18:06 ` Leo Famulari
2016-05-16 18:45 ` Leo Famulari
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160509033906.GC25977@jasmine \
--to=leo@famulari.name \
--cc=a@ajgrf.com \
--cc=guix-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).