GNU bug report logs - #54090
[PATCH 0/2] gnu: Add tessen.

Previous Next

Full log

View this message in rfc822 format

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Tanguy Le Carrour <tanguy <at> bioneland.org>
Cc: 54090 <at> debbugs.gnu.org, ludo <at> gnu.org, maximedevos <at> telenet.be
Subject: [bug#54090] [PATCH v2] gnu: Add tessen.
Date: Thu, 10 Mar 2022 19:56:19 +0100

Hello,

Tanguy Le Carrour <tanguy <at> bioneland.org> writes:

> * gnu/packages/password-utils.scm (tessen): New variable.

Thanks. I simply suggest some style overhaul!

> +    (arguments

(list
 #:phases
 #~(modify-phases ...)
 ...)

> +     `(#:phases (modify-phases %standard-phases
> +                  (add-after 'unpack 'patch-wtype-path
> +                    (lambda* (#:key inputs #:allow-other-keys)
> +                      (let ((wtype (assoc-ref inputs "wtype")))
> +                        (substitute* "tessen"
> +                          (("wtype") (string-append wtype "/bin/wtype"))))))

You can drop the `wtype' binding, and use

  (search-input-file inputs "/bin/wtype")

as a replacement for "wtype".

> +                  (delete 'configure)

There should be a comment (perhaps there is no configure script)

> +                  (delete 'check))

Disabling tests is done with "#:tests? #f", but then, a comment is
necessary (perhaps there are no tests).
)
> +       #:make-flags
> +       (let ((out (assoc-ref %outputs "out")))

  #~(let ...)

> +         (list (string-append "PREFIX=" out)))))

You can drop the `out' binding and use (string-append "PREFIX=" #$output)

You'll need to add

  #:use-module (guix gexp)

at the top-level for those changes.

Regards,
-- 
Nicolas Goaziou

Previous Next

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