GNU bug report logs - #36467
[PATCH 00/12] gnu: Add some Common Lisp libraries

Previous Next

Full log

Message #53 received at 36467 <at> debbugs.gnu.org (full text, mbox):

From: Guillaume Le Vaillant <glv <at> posteo.net>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: Andy Patterson <ajpatter <at> uwaterloo.ca>,
 Ludovic Courtès <ludo <at> gnu.org>,
 Katherine Cox-Buday <cox.katherine.e <at> gmail.com>, 36467 <at> debbugs.gnu.org
Subject: Re: [bug#36467] [PATCH v3] gnu: Add some Common Lisp libraries
Date: Mon, 16 Sep 2019 16:11:45 +0200

[Message part 1 (text/plain, inline)]

Pierre Neidhardt skribis:

> Hi Guillaume,
>
> thanks for the big patchset, looking very good overall!
>
> A few general nits:
>
> - You didn't define any "ecl-" package.  Could you define them for all the
>   packages (unless they are known to be broken with ECL)?
>   I know some packages lack the ecl- definition, this should be fixed :p
>
> - 6 commit messages lack a period on the first line.
>   If you use Emacs, you might find this yasnippet useful:
>   guix/etc/snippets/text-mode/guix-commit-message-add-package
>
> fprog:
>   - Description should start with a capital, or here maybe with @code{fprog}.
>
> cambl:
>   - is spelled in all-caps ("CAMBL") according to the README.
>
> cl-containers:
>   - synopsis: "Container library ..."
>
>   - I'm realizing the sbcl-lift does not have the right version number, which
>     should be (git-version "1.7.1" ...).  Can you update this?
>
> bst:
>   - 1.1 is out ;)
>
> cl-octet-streams:
>   - Description should start with a capital, or here maybe with @code{cl-octet-streams}.
>
> cl-lzlib
>   - License seems to be GPL-3, not GPL-3+, but since this is your library, the
>     decision is all yours ;)
>
> Thanks a lot for this great contribution to "Common Lisp on Guix" :)

Hi,

Here's the updated patch set (I think I didn't forget anything).

However some "ecl-" packages can't be built
right now because of a dependency that fails to build
(/gnu/store/sjyr2l309ldnfg9db718y585b3c13g51-ecl-hu.dwim.asdf-0.0.0-1.170b0e4.drv,
which apparently tries to load an SBCL-specific package).

[0001-gnu-Add-series.patch (text/x-diff, attachment)]

[0002-gnu-Add-periods.patch (text/x-diff, attachment)]

[0003-gnu-Add-periods-series.patch (text/x-diff, attachment)]

[0004-gnu-Add-metatilities-base.patch (text/x-diff, attachment)]

[0005-gnu-sbcl-lift-Fix-version-number.patch (text/x-diff, attachment)]

[0006-gnu-Add-cl-containers.patch (text/x-diff, attachment)]

[0007-gnu-Add-xlunit.patch (text/x-diff, attachment)]

[0008-gnu-Add-fprog.patch (text/x-diff, attachment)]

[0009-gnu-Add-cambl.patch (text/x-diff, attachment)]

[0010-gnu-Add-cl-ledger.patch (text/x-diff, attachment)]

[0011-gnu-Add-bst.patch (text/x-diff, attachment)]

[0012-gnu-Add-cl-octet-streams.patch (text/x-diff, attachment)]

[0013-gnu-Add-cl-lzlib.patch (text/x-diff, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.