[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Add guile-minikanren
From: |
Thompson, David |
Subject: |
Re: Add guile-minikanren |
Date: |
Thu, 23 Apr 2015 09:46:19 -0400 |
On Thu, Apr 23, 2015 at 9:17 AM, Andreas Enge <address@hidden> wrote:
> On Wed, Apr 22, 2015 at 10:15:27AM -0500, Christopher Allan Webber wrote:
>
>> I named it guile-minikanren which isn't really accurate. I'm not sure
>> how else I could name it though? I'd be open to suggestions!
>
> There is a chapter in the documentation about this:
> https://www.gnu.org/software/guix/manual/guix.html#Package-Naming
> The main idea is to not think too much, but to simply use the upstream
> project name. Here this seems to be "minikanren" without "guile-".
> We have special rules for perl and python; maybe we also need a special
> rule for guile?
The prefix "guile-" is important here because the build system
installs the Scheme modules into a directory that is specifically for
Guile, not other R6RS compliant Scheme implementations.
>
>> + (source (origin
>> + (method git-fetch)
>> + (uri (git-reference
>> + (url "https://github.com/ijp/minikanren.git")
>> + (commit "10d507785eab30b0f8b47bf8bb37d880731fc031")))
>
> Is there no tarball? If possiblem we would prefer this.
No tarball. I would recommend that the first 7 characters of the
commit SHA be used as the package version, and this string here could
just be replaced with 'version'.
>> + (native-inputs `(("guile" ,guile-2.0)))
>
> "native-inputs" are used during the build of the package, which is not
> the case here. Is guile needed at all as an input?
It will be needed. The build system should compile the Scheme files,
but it doesn't.
As a final note, I would like to add that the 'license' field can be
simply 'expat', instead of using the 'non-copyleft' procedure.
Thanks for working on this, Chris!
- Dave
- Add guile-minikanren, Christopher Allan Webber, 2015/04/22
- Re: Add guile-minikanren, Andreas Enge, 2015/04/23
- Re: Add guile-minikanren, Taylan Ulrich Bayırlı/Kammer, 2015/04/23
- Re: Add guile-minikanren,
Thompson, David <=
- Re: Add guile-minikanren, Andreas Enge, 2015/04/23
- Re: Add guile-minikanren, Ludovic Courtès, 2015/04/23
- Re: Add guile-minikanren, Eric Bavier, 2015/04/23
- Re: Add guile-minikanren, Andreas Enge, 2015/04/23
- Re: Add guile-minikanren, Ludovic Courtès, 2015/04/23
- Re: Add guile-minikanren, Christopher Allan Webber, 2015/04/23
- Re: Add guile-minikanren, Christopher Allan Webber, 2015/04/25
- Re: Add guile-minikanren, David Thompson, 2015/04/26