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

Previous Next

Full log

Message #49 received at 54090 <at> debbugs.gnu.org (full text, mbox):

From: Tanguy LE CARROUR <tanguy <at> bioneland.org>
To: 54090 <at> debbugs.gnu.org, Andrew Tropin <andrew <at> trop.in>
Cc: ludo <at> gnu.org, maximedevos <at> telenet.be, mail <at> nicolasgoaziou.fr
Subject: Re: [bug#54090] [PATCH v3] gnu: Add tessen.
Date: Fri, 25 Mar 2022 17:22:25 +0100

Hi Andrew,


Quoting Andrew Tropin (2022-03-25 16:10:25)
> On 2022-03-11 08:54, Tanguy Le Carrour wrote:
> 
> > * gnu/packages/password-utils.scm (tessen): New variable.
> > […]
> > +    (arguments
> > +     (list #:tests?
> > +           #f ;no tests
> > +           #:phases
> > +           #~(modify-phases %standard-phases
> > +               (add-after 'unpack 'patch-wtype-path
> > +                 (lambda* (#:key inputs #:allow-other-keys)
> > +                   (substitute* "tessen"
> > +                     (("wtype") (search-input-file inputs "/bin/wtype")))))
> > +               (delete 'configure)) ;no configure script
> > +           #:make-flags
> > +           #~(list (string-append "PREFIX="
> > […]
> 
> Sorry, for comming up late, but the path to wl-copy binary should be
> hardcoded as well.
> 
> /gnu/store/f48prcjwpw8wrfz0srjf096a9s8fja2w-profile/bin/tessen: line 476: wl-copy: command not found

Oh, my bad! I'm not using the "copy to clipboard" feature. Fortunatly,
the patch is trivial. But, before submitting it, I have a question!…

Why not also add as inputs and substitute `pass`, `gopass`, `wofi`,
`notify-send`, `xdg-open`… that are also used by tessen?!

Regards,

-- 
Tanguy

Previous Next

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