[bug#67123] [PATCH] gnu: Add pass-coffin

[bug#67123] [PATCH] gnu: Add pass-coffin

[Christian Miller via Guix-patches via]

* gnu/packages/password-utils.scm (pass-coffin): New variable.

Change-Id: I2fdf6c755b9b75780c5a3c0304e5b961a15640a4
---
 gnu/packages/password-utils.scm | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/gnu/packages/password-utils.scm b/gnu/packages/password-utils.scm
index 385bd64985..f9767b91af 100644
--- a/gnu/packages/password-utils.scm
+++ b/gnu/packages/password-utils.scm
@@ -39,6 +39,7 @@
 ;;; Copyright © 2022 ( <paren@disroot.org>
 ;;; Copyright © 2022 Nicolas Graves <ngraves@ngraves.fr>
 ;;; Copyright © 2022 Petr Hodina <phodina@protonmail.com>
+;;; Copyright © 2023 Christian Miller <christian.miller@dadoes.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -1567,6 +1568,37 @@ (define-public pass-tomb
 pass-tomb to automatically close your store after a given time.")
     (license license:gpl3+)))
 
+(define-public pass-coffin
+  (package
+    (name "pass-coffin")
+    (version "1.2.1")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/ayushnix/pass-coffin")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "1486ikwsdjsj74qf949vk47r8mfp2mbbdc3scs8786nnnkhzc89n"))))
+    (build-system gnu-build-system)
+    (arguments
+     (list
+      #:tests? #f ;No tests
+      #:make-flags #~(list (string-append "PREFIX="
+                                          #$output)
+                           (string-append "BASHCOMPDIR="
+                                          #$output "/etc/bash_completion.d"))
+      #:phases #~(modify-phases %standard-phases
+                   (delete 'configure))))
+    (inputs (list password-store tar))
+    (home-page "https://github.com/ayushnix/pass-coffin")
+    (synopsis "Pass extension to keep the tree of passwords encrypted")
+    (description
+     "Pass-coffin is a pass extension that hides the password store
+data inside a GPG encrypted file, which we'll call a coffin.")
+    (license license:gpl3)))
+
 (define-public xkcdpass
   (package
     (name "xkcdpass")

base-commit: af6105afc67a15a491a0a4fd18a28c9f801a0b94
-- 
2.41.0

[bug#67123] [PATCH v2 0/1] gnu: Add pass-coffin

[Steve George]

Review by Steve George: 
  * submission: checked commit format, synopsis and commit
  * submission: license, patch applies cleanly
  * functionality: test installed, guix lint is clean
  * code: checked style
  * added: Reviewed-by commit trailer
  * re-roll to trigger QA build

  * Question for committer: this is an add-on to password-store and can't be
    used without it. Should password-store be a propagated-input?


Christian Miller via Guix-patches via (1):
  gnu: Add pass-coffin

 gnu/packages/password-utils.scm | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)


base-commit: e7990b12170eec0c35541196f948be26a9e4a519
-- 
2.41.0

[bug#67123] [PATCH v2 1/1] gnu: Add pass-coffin

[Steve George]

From: Christian Miller via Guix-patches via <guix-patches@gnu.org>

* gnu/packages/password-utils.scm (pass-coffin): New variable.

Reviewed-by: Steve George <steve@futurile.net>

Change-Id: I2fdf6c755b9b75780c5a3c0304e5b961a15640a4
---
 gnu/packages/password-utils.scm | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/gnu/packages/password-utils.scm b/gnu/packages/password-utils.scm
index b79fdfc6a0f..09be72cfa6b 100644
--- a/gnu/packages/password-utils.scm
+++ b/gnu/packages/password-utils.scm
@@ -39,6 +39,7 @@
 ;;; Copyright © 2022 ( <paren@disroot.org>
 ;;; Copyright © 2022 Nicolas Graves <ngraves@ngraves.fr>
 ;;; Copyright © 2022 Petr Hodina <phodina@protonmail.com>
+;;; Copyright © 2023 Christian Miller <christian.miller@dadoes.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -1562,6 +1563,37 @@ (define-public pass-tomb
 pass-tomb to automatically close your store after a given time.")
     (license license:gpl3+)))
 
+(define-public pass-coffin
+  (package
+    (name "pass-coffin")
+    (version "1.2.1")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/ayushnix/pass-coffin")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "1486ikwsdjsj74qf949vk47r8mfp2mbbdc3scs8786nnnkhzc89n"))))
+    (build-system gnu-build-system)
+    (arguments
+     (list
+      #:tests? #f ;No tests
+      #:make-flags #~(list (string-append "PREFIX="
+                                          #$output)
+                           (string-append "BASHCOMPDIR="
+                                          #$output "/etc/bash_completion.d"))
+      #:phases #~(modify-phases %standard-phases
+                   (delete 'configure))))
+    (inputs (list password-store tar))
+    (home-page "https://github.com/ayushnix/pass-coffin")
+    (synopsis "Pass extension to keep the tree of passwords encrypted")
+    (description
+     "Pass-coffin is a pass extension that hides the password store
+data inside a GPG encrypted file, which we'll call a coffin.")
+    (license license:gpl3)))
+
 (define-public xkcdpass
   (package
     (name "xkcdpass")
-- 
2.41.0

[bug#67123] [PATCH v2 0/1] gnu: Add pass-coffin

[Clément Lassieur]

On Sat, Feb 24 2024, Steve George wrote:

> Review by Steve George: 
>   * submission: checked commit format, synopsis and commit
>   * submission: license, patch applies cleanly
>   * functionality: test installed, guix lint is clean
>   * code: checked style
>   * added: Reviewed-by commit trailer
>   * re-roll to trigger QA build
>
>   * Question for committer: this is an add-on to password-store and can't be
>     used without it. Should password-store be a propagated-input?

I don't think so.  See emacs for example.  That would make it harder to
use another version of password-store.

> Christian Miller via Guix-patches via (1):
>   gnu: Add pass-coffin
>
>  gnu/packages/password-utils.scm | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>
>
> base-commit: e7990b12170eec0c35541196f948be26a9e4a519

bug#67123: Close

[Andreas Enge]

Hello,

like Clément, I would say we should not propagate password-store.
It would also create a weird situation when people install password-store
with one revision of Guix, and add this extension from a different revision.
More generally, password-store in a profile and from the propagated input
could clash, and it would be surprising if installing an extension swaps
the main program under your feet.

Applied, thanks for the patch and the review!

Andreas