GNU bug report logs - #67917
[PATCH 0/2] guix import cpan improvements

Previous Next

Package: guix-patches;

Reported by: Wilko Meyer <w <at> wmeyer.eu>

Date: Tue, 19 Dec 2023 23:44:01 UTC

Severity: normal

Tags: patch

Full log


View this message in rfc822 format

From: Wilko Meyer <w <at> wmeyer.eu>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: Wilko Meyer <w <at> wmeyer.eu>, 67917 <at> debbugs.gnu.org
Subject: [bug#67917] [PATCH 0/2] guix import cpan improvements
Date: Sat, 23 Dec 2023 12:49:07 +0100
Hi Ludo,

Thanks for reviewing this so quickly!

Ludovic Courtès <ludo <at> gnu.org> writes:

> I think this doesn’t belong here: right now, ‘define-public’ is added
> when using ‘-r’ by ‘package->definition’ in (guix import utils).  This
> is shared by most importers so we cannot just change it in a single
> importer.

This currently seems to be inconsistent among importers. crate and go
always add define-public independent of using '-r', while e.g. egg, gnu,
elpa etc. do not do that. I'll prepare a v2 of this patch series without
this change, as the cpan importer should stay at its default behaviour.

Speaking of '-r', the cpan importer doesn't offer recursively importing
packages. I could add this functionality to the cpan importer in v2 of
this patch series, WDYT?

> This one I’m not sure, but we should make sure importers are consistent
> as well.

Agreed.

-- 
Kind regards,

Wilko Meyer
w <at> wmeyer.eu




This bug report was last modified 1 year and 159 days ago.

Previous Next


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