* [bug#58397] [PATCH 0/2] gnu: emacs-password-store: Refer directly to password-store.
@ 2022-10-09 16:56 ( via Guix-patches via
2022-10-09 16:57 ` [bug#58397] [PATCH 1/2] " ( via Guix-patches via
0 siblings, 1 reply; 6+ messages in thread
From: ( via Guix-patches via @ 2022-10-09 16:56 UTC (permalink / raw)
To: 58397; +Cc: (
This patchset removes a propagated external command, which, as we all know, are
malicious parasites that have no purpose in life but undermining the benefits of
Guix.
( (2):
gnu: emacs-password-store: Refer directly to password-store.
gnu: emacs-ivy-pass: Don't propagate password-store.
gnu/packages/emacs-xyz.scm | 64 +++++++++++++++++++++-----------------
1 file changed, 36 insertions(+), 28 deletions(-)
--
2.38.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#58397] [PATCH 1/2] gnu: emacs-password-store: Refer directly to password-store.
2022-10-09 16:56 [bug#58397] [PATCH 0/2] gnu: emacs-password-store: Refer directly to password-store ( via Guix-patches via
@ 2022-10-09 16:57 ` ( via Guix-patches via
2022-10-09 16:57 ` [bug#58397] [PATCH 2/2] gnu: emacs-ivy-pass: Don't propagate password-store ( via Guix-patches via
2022-10-10 16:12 ` bug#58397: [PATCH 1/2] gnu: emacs-password-store: Refer directly to password-store Nicolas Goaziou
0 siblings, 2 replies; 6+ messages in thread
From: ( via Guix-patches via @ 2022-10-09 16:57 UTC (permalink / raw)
To: 58397; +Cc: (
* gnu/packages/emacs-xyz.scm (emacs-password-store)[arguments]:
Use gexp style.
<#:phases>{'extract-el-file}: New phase.
[propagated-inputs]<password-store>: Move it...
[inputs]: ...here.
---
gnu/packages/emacs-xyz.scm | 35 ++++++++++++++++++++++-------------
1 file changed, 22 insertions(+), 13 deletions(-)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 8134a19d56..6f415dc7b1 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -18918,24 +18918,33 @@ (define-public emacs-password-store
(uri (git-reference
(url "git://git.zx2c4.com/password-store")
(commit commit)))
+ (file-name (git-file-name name version))
(sha256
(base32
- "0ni62f4pq96g0i0q66bch1dl9k4zqwhg7xaf746k3gbbqxcdh3vi"))
- (file-name (git-file-name name version))))
+ "0ni62f4pq96g0i0q66bch1dl9k4zqwhg7xaf746k3gbbqxcdh3vi"))))
(build-system emacs-build-system)
(arguments
- `(#:phases
- (modify-phases %standard-phases
- (add-after 'unpack 'extract-el-file
- (lambda _
- (copy-file "contrib/emacs/password-store.el" "password-store.el")
- (delete-file-recursively "contrib")
- (delete-file-recursively "man")
- (delete-file-recursively "src")
- (delete-file-recursively "tests"))))))
+ (list #:phases
+ #~(modify-phases %standard-phases
+ (add-after 'unpack 'extract-el-file
+ (lambda _
+ (copy-file "contrib/emacs/password-store.el"
+ "password-store.el")
+ (delete-file-recursively "contrib")
+ (delete-file-recursively "man")
+ (delete-file-recursively "src")
+ (delete-file-recursively "tests")))
+ (add-after 'extract-el-file 'patch-executables
+ (lambda* (#:key inputs #:allow-other-keys)
+ (substitute* "password-store.el"
+ (("\\(executable-find \"pass\"\\)")
+ (string-append
+ "\""
+ (search-input-file inputs "bin/pass")
+ "\""))))))))
+ (inputs (list password-store))
(propagated-inputs
- (list emacs-auth-source-pass emacs-s emacs-with-editor
- password-store))
+ (list emacs-auth-source-pass emacs-s emacs-with-editor))
(home-page "https://git.zx2c4.com/password-store/tree/contrib/emacs")
(synopsis "Password store (pass) support for Emacs")
(description
--
2.38.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* [bug#58397] [PATCH 2/2] gnu: emacs-ivy-pass: Don't propagate password-store.
2022-10-09 16:57 ` [bug#58397] [PATCH 1/2] " ( via Guix-patches via
@ 2022-10-09 16:57 ` ( via Guix-patches via
2022-10-10 16:27 ` Nicolas Goaziou
2022-10-10 16:12 ` bug#58397: [PATCH 1/2] gnu: emacs-password-store: Refer directly to password-store Nicolas Goaziou
1 sibling, 1 reply; 6+ messages in thread
From: ( via Guix-patches via @ 2022-10-09 16:57 UTC (permalink / raw)
To: 58397; +Cc: (
* gnu/packages/emacs-xyz.scm (emacs-ivy-pass)[propagated-inputs]
<password-store>: Delete it.
[synopsis, description]: Use @command when referring to pass.
---
gnu/packages/emacs-xyz.scm | 29 ++++++++++++++---------------
1 file changed, 14 insertions(+), 15 deletions(-)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 6f415dc7b1..27494f2a37 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -10535,23 +10535,22 @@ (define-public emacs-ivy-pass
(package
(name "emacs-ivy-pass")
(version (git-version "0.1" revision commit))
- (source
- (origin
- (method git-fetch)
- (uri (git-reference
- (url "https://github.com/ecraven/ivy-pass")
- (commit commit)))
- (file-name (git-file-name name version))
- (sha256
- (base32
- "18crb4zh2pjf0cmv3b913m9vfng27girjwfqc3mk7vqd1r5a49yk"))))
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/ecraven/ivy-pass")
+ (commit commit)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32
+ "18crb4zh2pjf0cmv3b913m9vfng27girjwfqc3mk7vqd1r5a49yk"))))
(build-system emacs-build-system)
- (propagated-inputs
- (list emacs-ivy emacs-password-store password-store))
+ (propagated-inputs (list emacs-ivy emacs-password-store))
(home-page "https://github.com/ecraven/ivy-pass")
- (synopsis "Ivy interface for password store (pass)")
- (description "This package provides an Ivy interface for working with
-the password store @code{pass}.")
+ (synopsis "Ivy interface to @code{pass}")
+ (description
+ "This package provides an Ivy interface for working with the
+password store (@command{pass}) in Emacs.")
(license license:gpl3))))
(define-public emacs-ivy-yasnippet
--
2.38.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* [bug#58397] [PATCH 2/2] gnu: emacs-ivy-pass: Don't propagate password-store.
2022-10-09 16:57 ` [bug#58397] [PATCH 2/2] gnu: emacs-ivy-pass: Don't propagate password-store ( via Guix-patches via
@ 2022-10-10 16:27 ` Nicolas Goaziou
0 siblings, 0 replies; 6+ messages in thread
From: Nicolas Goaziou @ 2022-10-10 16:27 UTC (permalink / raw)
To: 58397; +Cc: (, 58397-done
Hello,
"( via Guix-patches" via <guix-patches@gnu.org> writes:
> * gnu/packages/emacs-xyz.scm (emacs-ivy-pass)[propagated-inputs]
> <password-store>: Delete it.
> [synopsis, description]: Use @command when referring to pass.
Thank you. Applied with the following changes.
> - (source
> - (origin
> - (method git-fetch)
> - (uri (git-reference
> - (url "https://github.com/ecraven/ivy-pass")
> - (commit commit)))
> - (file-name (git-file-name name version))
> - (sha256
> - (base32
> - "18crb4zh2pjf0cmv3b913m9vfng27girjwfqc3mk7vqd1r5a49yk"))))
> + (source (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/ecraven/ivy-pass")
> + (commit commit)))
> + (file-name (git-file-name name version))
> + (sha256
> + (base32
> + "18crb4zh2pjf0cmv3b913m9vfng27girjwfqc3mk7vqd1r5a49yk"))))
This change is not warranted, so I removed it.
> (build-system emacs-build-system)
> - (propagated-inputs
> - (list emacs-ivy emacs-password-store password-store))
> + (propagated-inputs (list emacs-ivy emacs-password-store))
I think the newline removal isn't either.
> (home-page "https://github.com/ecraven/ivy-pass")
> - (synopsis "Ivy interface for password store (pass)")
> - (description "This package provides an Ivy interface for working with
> -the password store @code{pass}.")
> + (synopsis "Ivy interface to @code{pass}")
> + (description
> + "This package provides an Ivy interface for working with the
> +password store (@command{pass}) in Emacs.")
I used @code here too, since you're probably referring to the project's
name.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#58397: [PATCH 1/2] gnu: emacs-password-store: Refer directly to password-store.
2022-10-09 16:57 ` [bug#58397] [PATCH 1/2] " ( via Guix-patches via
2022-10-09 16:57 ` [bug#58397] [PATCH 2/2] gnu: emacs-ivy-pass: Don't propagate password-store ( via Guix-patches via
@ 2022-10-10 16:12 ` Nicolas Goaziou
2022-10-10 16:23 ` [bug#58397] " ( via Guix-patches via
1 sibling, 1 reply; 6+ messages in thread
From: Nicolas Goaziou @ 2022-10-10 16:12 UTC (permalink / raw)
To: ( via Guix-patches via; +Cc: (, 58397-done
Hello,
"( via Guix-patches" via <guix-patches@gnu.org> writes:
Thank you. Applied with the following change.
> + (add-after 'extract-el-file 'patch-executables
> + (lambda* (#:key inputs #:allow-other-keys)
> + (substitute* "password-store.el"
> + (("\\(executable-find \"pass\"\\)")
> + (string-append
> + "\""
> + (search-input-file inputs "bin/pass")
> + "\""))))))))
I used `emacs-substitute-variable' instead of substitute*.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-10-10 16:33 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-09 16:56 [bug#58397] [PATCH 0/2] gnu: emacs-password-store: Refer directly to password-store ( via Guix-patches via
2022-10-09 16:57 ` [bug#58397] [PATCH 1/2] " ( via Guix-patches via
2022-10-09 16:57 ` [bug#58397] [PATCH 2/2] gnu: emacs-ivy-pass: Don't propagate password-store ( via Guix-patches via
2022-10-10 16:27 ` Nicolas Goaziou
2022-10-10 16:12 ` bug#58397: [PATCH 1/2] gnu: emacs-password-store: Refer directly to password-store Nicolas Goaziou
2022-10-10 16:23 ` [bug#58397] " ( via Guix-patches via
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).