unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / Atom feed
* [bug#46646] [PATCH] gnu: Add pwsafe-cli.
@ 2021-02-19 21:16 Stefan Reichör
  2021-02-23 17:34 ` bug#46646: Léo Le Bouter via Guix-patches via
  2021-02-23 17:54 ` [bug#46646] [PATCH] gnu: Add pwsafe-cli Nicolas Goaziou
  0 siblings, 2 replies; 5+ messages in thread
From: Stefan Reichör @ 2021-02-19 21:16 UTC (permalink / raw)
  To: 46646; +Cc: Stefan Reichör

* gnu/packages/password-utils.scm (pwsafe-cli): New variable.
---
 gnu/packages/password-utils.scm | 44 +++++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)

diff --git a/gnu/packages/password-utils.scm b/gnu/packages/password-utils.scm
index ef4bf535a4..6d6834cfd5 100644
--- a/gnu/packages/password-utils.scm
+++ b/gnu/packages/password-utils.scm
@@ -29,6 +29,7 @@
 ;;; Copyright © 2020 Jean-Baptiste Note <jean-baptiste.note@m4x.org>
 ;;; Copyright © 2020 Michael Rohleder <mike@rohleder.de>
 ;;; Copyright © 2020 Vinicius Monego <monego@posteo.net>
+;;; Copyright © 2021 Stefan Reichör <stefan@xsteve.at>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -58,6 +59,7 @@
   #:use-module (gnu packages admin)
   #:use-module (gnu packages aidc)
   #:use-module (gnu packages authentication)
+  #:use-module (gnu packages autotools)
   #:use-module (gnu packages base)
   #:use-module (gnu packages check)
   #:use-module (gnu packages compression)
@@ -255,6 +257,48 @@ There are other programs that support the file format on different
 platforms.")
     (license license:artistic2.0)))
 
+(define-public pwsafe-cli
+  (let ((commit "c49a0541b66647ad04d19ddb351d264054c67759")
+        (revision "0"))
+    (package
+      (name "pwsafe-cli")
+      (version (git-version "0" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/nsd20463/pwsafe")
+               (commit commit)))
+         (sha256
+          (base32
+           "0ak09r1l7k57m6pdx468hhzvz0szmaq42vyr575fvsjc8rbrp8qq"))
+         (file-name (git-file-name name version))))
+      (build-system gnu-build-system)
+      (arguments
+       `(#:tests? #f ; skip failing test suite (requires write access to /tmp)
+         #:phases
+         (modify-phases %standard-phases
+         (replace 'bootstrap
+           (lambda _
+             (invoke "aclocal")
+             (invoke "autoheader")
+             (invoke "automake" "--add-missing")
+             (invoke "autoconf")
+             #t)))))
+      (native-inputs
+       `(("autoconf" ,autoconf)
+         ("automake" ,automake)))
+      (inputs
+       `(("openssl" ,openssl)
+         ("libx11" ,libx11)
+         ("libxt" ,libxt)
+         ("libxmu" ,libxmu)))
+      (home-page "https://github.com/nsd20463/pwsafe")
+      (synopsis "CLI password manager")
+      (description "Command line tool compatible with Counterpane's
+Passwordsafe.")
+      (license license:gpl2))))
+
 (define-public shroud
   (package
     (name "shroud")
-- 
2.25.1





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#46646:
  2021-02-19 21:16 [bug#46646] [PATCH] gnu: Add pwsafe-cli Stefan Reichör
@ 2021-02-23 17:34 ` Léo Le Bouter via Guix-patches via
  2021-02-23 17:54 ` [bug#46646] [PATCH] gnu: Add pwsafe-cli Nicolas Goaziou
  1 sibling, 0 replies; 5+ messages in thread
From: Léo Le Bouter via Guix-patches via @ 2021-02-23 17:34 UTC (permalink / raw)
  To: 46646-done

[-- Attachment #1: Type: text/plain, Size: 343 bytes --]

Pushed as 33a212bf88904295ddc20fcb05a4c237205812f8

I tried to enable the tests by substituting /tmp in all files in the
"test" directory but somehow the files created by the tests in a
writable directory are not writable themselves, still quite obscure to
me.

Please suggest another patch if you fix the tests one day.

Thank you!

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#46646] [PATCH] gnu: Add pwsafe-cli.
  2021-02-19 21:16 [bug#46646] [PATCH] gnu: Add pwsafe-cli Stefan Reichör
  2021-02-23 17:34 ` bug#46646: Léo Le Bouter via Guix-patches via
@ 2021-02-23 17:54 ` Nicolas Goaziou
  2021-02-23 18:04   ` Stefan Reichör
  1 sibling, 1 reply; 5+ messages in thread
From: Nicolas Goaziou @ 2021-02-23 17:54 UTC (permalink / raw)
  To: Stefan Reichör; +Cc: 46646

Hello,

Stefan Reichör <stefan@xsteve.at> writes:

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

Thanks!

 The project seems to be named "pwsafe", and all other distributions use
"pwsafe" for it. I understand it is a CLI, but is there a particular
reason to name it "pwsafe-cli"?

Regards,
-- 
Nicolas Goaziou




^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#46646] [PATCH] gnu: Add pwsafe-cli.
  2021-02-23 17:54 ` [bug#46646] [PATCH] gnu: Add pwsafe-cli Nicolas Goaziou
@ 2021-02-23 18:04   ` Stefan Reichör
  2021-02-23 18:20     ` Nicolas Goaziou
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Reichör @ 2021-02-23 18:04 UTC (permalink / raw)
  To: Nicolas Goaziou; +Cc: 46646

Hi Nicolas!

> Hello,
>
> Stefan Reichör <stefan@xsteve.at> writes:
>
>> * gnu/packages/password-utils.scm (pwsafe-cli): New variable.
>
> Thanks!
>
>  The project seems to be named "pwsafe", and all other distributions use
> "pwsafe" for it. I understand it is a CLI, but is there a particular
> reason to name it "pwsafe-cli"?
>
> Regards,

I would also prefer pwsafe as name, however guix already has a package named pwsafe.
This is why I picked a different name.

I would have no problem in renaming pwsafe to something else and name pwsafe-cli pwsafe... ;-)

Stefan.





^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#46646] [PATCH] gnu: Add pwsafe-cli.
  2021-02-23 18:04   ` Stefan Reichör
@ 2021-02-23 18:20     ` Nicolas Goaziou
  0 siblings, 0 replies; 5+ messages in thread
From: Nicolas Goaziou @ 2021-02-23 18:20 UTC (permalink / raw)
  To: Stefan Reichör; +Cc: 46646

Stefan Reichör <stefan@xsteve.at> writes:

> I would also prefer pwsafe as name, however guix already has a package named pwsafe.
> This is why I picked a different name.

Ah true! I hadn't realized this.

> I would have no problem in renaming pwsafe to something else and name
> pwsafe-cli pwsafe... ;-)

And we also have passwordsafe…

According to other distributions, we should do the following renaming

  passwordsafe -> gnome-passwordsafe
  pwsafe       -> passwordsafe
  pwsafe-cli   -> pwsafe

I guess that ship has sailed…

Thank you for your answer!

Regards,




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-02-23 18:21 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-19 21:16 [bug#46646] [PATCH] gnu: Add pwsafe-cli Stefan Reichör
2021-02-23 17:34 ` bug#46646: Léo Le Bouter via Guix-patches via
2021-02-23 17:54 ` [bug#46646] [PATCH] gnu: Add pwsafe-cli Nicolas Goaziou
2021-02-23 18:04   ` Stefan Reichör
2021-02-23 18:20     ` Nicolas Goaziou

unofficial mirror of guix-patches@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-patches/1 guix-patches/git/1.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-patches guix-patches/ https://yhetil.org/guix-patches \
		guix-patches@gnu.org
	public-inbox-index guix-patches

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.patches


AGPL code for this site: git clone http://ou63pmih66umazou.onion/public-inbox.git