guix-patches
[Top][All Lists]
Advanced

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

[bug#32483] [PATCH] import: stackage: Support recursive importing.


From: Ludovic Courtès
Subject: [bug#32483] [PATCH] import: stackage: Support recursive importing.
Date: Thu, 23 Aug 2018 17:03:02 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Hello!

Ricardo Wurmus <address@hidden> skribis:

> * guix/import/hackage.scm (hackage-name->package-name): Export procedure.
> * guix/import/stackage.scm (lts-info-packages-lts-info): Fix match expression.
> (stackage-recursive-import): New procedure.
> (stackage->guix-package): Memoize results.
> * guix/scripts/import/stackage.scm (show-help, %options,
> guix-import-stackage): Support recursive importing.
> * doc/guix.texi (Invoking guix import): Document option.

Could you add a test for recursive imports, similar to what Oleg did in
tests/gem.scm?  Or perhaps that’d be redundant since it does little more
than call ‘recursive-import’ from (guix import utils)?  (And I realize
tests/elpa.scm doesn’t have recursive import tests.)

Anyway, apart from that it LGTM.  Thanks!

Ludo’.







reply via email to

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