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: Christopher Baines <mail <at> cbaines.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 49169 <at> debbugs.gnu.org
Subject: [bug#49169] [PATCH 00/11] Removing input labels from package definitions
Date: Sun, 27 Jun 2021 19:37:26 +0100
[Message part 1 (text/plain, inline)]
Ludovic Courtès <ludo <at> gnu.org> writes:

> Ludovic Courtès <ludo <at> gnu.org> skribis:
>
>> The main change is the addition of ‘guix style’, based on the script
>> I posted earlier.  ‘guix style’ is able to systematically preserve
>> comments (margin comments and line comments).  It recognizes and
>> “translates” several common idioms.
>
> Here’s a couple of diffs generated by ‘guix style’:
>
> @@ -171,9 +170,9 @@ engine programmable using Lua.")
>      (inherit arcan)
>      (name "arcan-sdl")
>      (inputs
> -     `(("sdl" ,sdl)
> -       ,@(fold alist-delete (package-inputs arcan)
> -               '("libdrm"))))
> +     (modify-inputs (package-inputs arcan)
> +       (delete "libdrm")
> +       (prepend sdl)))
>      (arguments
>       `(,@(ensure-keyword-arguments
>            (package-arguments arcan)
> @@ -218,11 +217,7 @@ engine programmable using Lua.")
>             ,(string-append "--with-xkb-output="
>                             "/tmp"))))   ; FIXME: Copied from xorg
>        (native-inputs
> -       `(("pkg-config" ,pkg-config)
> -         ("autoconf" ,autoconf)
> -         ("automake" ,automake)
> -         ("libtool" ,libtool)
> -         ("util-macros" ,util-macros)))
> +       (list pkg-config autoconf automake libtool util-macros))
>        (inputs
>         `(("arcan" ,arcan)
>           ("font-util" ,font-util)
>
> Comments:
>
> @@ -3797,11 +3713,11 @@ important tasks for becoming a daemon process:
>                  "1aycpc387wqz7h9w2p53qxn43qsh3m6by6ak4kkc66x9aprr63rz"))))
>      (build-system python-build-system)
>      (propagated-inputs
> -     `(("python-six" ,python-six)))
> +     (list python-six))
>      (native-inputs
> -     `(;; For tests.
> -       ("graphviz" ,graphviz)           ;for 'dot'
> -       ("python-nose" ,python-nose)))
> +     (list ;; For tests.
> +           graphviz ;for 'dot'
> +           python-nose))
>      (home-page "https://github.com/c0fec0de/anytree")
>      (synopsis "Lightweight tree data library")
>
> Pretty cool, no?  :-)

The automated translation is pretty impressive! I've had a quick look
through the patches and they look good to me.
[signature.asc (application/pgp-signature, inline)]

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.