[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’.