Subject: Re: bug#54090: [PATCH 0/2] gnu: Add tessen.

From: Ludovic Courtès <[email-masked]>
Date: Thu, 03 Mar 2022 23:30:23 +0100

Tanguy Le Carrour <[email-masked]> skribis:

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

[...]

> +     (origin
> +       (method url-fetch)
> +       (uri
> +        (string-append "https://raw.githubusercontent.com/ayushnix/tessen/"
> +                       "v" version "/tessen"))

Is this a stable URL?

> +       #:builder
> +       (begin
> +         (use-modules (guix build utils))
> +         (let ((source (string-append (assoc-ref %build-inputs "source")))
> +               (script "tessen")
> +               (out (assoc-ref %outputs "out")))

Could you change that to use a gexp, along these lines:

  #:builder #~(begin
                …
                (let ((source #$(package-source this-package))
                      (out    #$output)
                      …)
                  …))

> +    (propagated-inputs
> +     (list wtype))

Could you substitute* the absolute file name of ‘wtype’ in the script
instead of propagating?

TIA!

Ludo’.