GNU bug report logs - #49169
[PATCH 00/11] Removing input labels from package definitions

Previous Next

Package: guix-patches;

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

Date: Tue, 22 Jun 2021 09:03:01 UTC

Severity: normal

Tags: patch

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

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Ludovic Courtès <ludo <at> gnu.org>
To: Sarah Morgensen <iskarian <at> mgsn.dev>
Cc: 49169 <at> debbugs.gnu.org
Subject: [bug#49169] [PATCH 00/11] Removing input labels from package definitions
Date: Sun, 11 Jul 2021 01:15:33 +0200
Hi again,

Sarah Morgensen <iskarian <at> mgsn.dev> skribis:

> diff --git a/guix/import/utils.scm b/guix/import/utils.scm
> index d817318a91..5075e5c491 100644
> --- a/guix/import/utils.scm
> +++ b/guix/import/utils.scm
> @@ -237,12 +237,10 @@ into a proper sentence and by using two spaces between sentences."
>  optional OUTPUT, tries to generate a quoted list of inputs, as suitable to
>  use in an 'inputs' field of a package definition."
>    (define (make-input input version)
> -    (cons* input (list 'unquote (string->symbol
> -                                 (if version
> -                                     (string-append input "-" version)
> -                                     input)))
> -           (or (and output (list output))
> -               '())))
> +    (let ((name (if version (string-append input "-" version) input)))
> +      (if output
> +          (list (string->symbol name) output)
> +          (string->symbol name))))
>  
>    (map (match-lambda
>           ((input version) (make-input input version))
> @@ -263,7 +261,7 @@ snippet generated is for regular inputs."
>        (()
>         '())
>        ((package-inputs ...)
> -       `((,field-name (,'quasiquote ,package-inputs)))))))
> +       `((,field-name (list ,@package-inputs)))))))
>  
>  (define* (maybe-native-inputs package-names #:optional (output #f))
>    "Same as MAYBE-INPUTS, but for native inputs."

On closer inspection, it seems that this change would affect the Crate
importer.  Unfortunately, Crate packages live in their own world and are
unaffected by package input simplification.

This is unfortunate, also because they probably never even use input
labels that can be seen in #:cargo-development-inputs and the likes.

That said, it would be good to simplify that too, and I’m open to
suggestions on how to achieve that!

Ludo’.




This bug report was last modified 3 years and 352 days ago.

Previous Next


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