guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#31399] [PATCH] import: elpa: Implement recursive import.


From: Ricardo Wurmus
Subject: [bug#31399] [PATCH] import: elpa: Implement recursive import.
Date: Fri, 25 May 2018 14:50:30 +0200
User-agent: mu4e 1.0; emacs 25.3.1

Hi,

Ludovic Courtès <address@hidden> writes:

> Oleg Pykhalov <address@hidden> skribis:
>
>> I stole recursive importer from ‘cran’ and made it for ‘elpa’, the patch
>> is attached.  I don't like it and want to share the code with ‘cran’
>> importer, but I don't know how to do it without increasing complexity.
>
> Indeed.  Ricardo and I discussed this in the past and Ricardo started
> implementing something:
>
>   http://lists.gnu.org/archive/html/guix-devel/2016-08/msg00381.html
>
> It would be great to resume work on this, especially since we have other
> importers that would immediately benefit from it (CRAN, Go, Crates).
>
> The solution I proposed back then was to have importers return a SRFI-41
> lazy stream of package definitions.

The CRAN importer supports recursion and it is implemented with SRFI-41
lazy streams and AIUI this is what this patch does for the elpa
importer.

We should move “recursive-import” to “(guix import utils)” and let it
take the package generator procedure (e.g. “cran->guix-package”) as an
argument.  It looks like this patch copied “recursive-import” and
changed little more than “cran->guix-package” to “elpa->guix-package”,
so this seems like a simple change.

--
Ricardo





reply via email to

[Prev in Thread] Current Thread [Next in Thread]