[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#36052: 26.2.50; [PATCH] Improve auth-source-pass
From: |
Eli Zaretskii |
Subject: |
bug#36052: 26.2.50; [PATCH] Improve auth-source-pass |
Date: |
Sat, 08 Jun 2019 19:02:54 +0300 |
> From: Damien Cassou <damien@cassou.me>
> Date: Sat, 08 Jun 2019 17:47:53 +0200
> Cc: Magnus Henoch <magnus.henoch@gmail.com>, Nicolas Petton
> <nicolas@petton.fr>,
> Iku Iwasa <iku.iwasa@gmail.com>, Keith Amidon <camalot@picnicpark.org>,
> galaunay <gaby.launay@tutanota.com>, 36052@debbugs.gnu.org,
> Ted Zlatanov <tzz@lifelogs.com>
>
> +(defcustom auth-source-pass-filename "~/.password-store"
> + "Path to the password-store folder."
> + :type 'directory)
Please add/update a :version tag to any defcustom that you introduce
or modify. This is the basis for a "C-h v" and a few other features
that report on options introduced/changed in a certain Emacs version.
Also, GNU Coding Standards frown on calling "path" anything that is
not colon-separated PATH-style list of directories.
> >From d46e8dd1bbc3a1c7ac6506e3e6cfe3d87e57e99e Mon Sep 17 00:00:00 2001
> From: galaunay <gaby.launay@tutanota.com>
> Date: Sun, 13 Jan 2019 21:30:53 +0000
> Subject: [PATCH 04/12] Add auth-source-pass-path option
This patch was included twice, it seems.
> +(defun auth-source-pass--get-attr (key entry-data)
> + "Return the value associated to KEY in data from an already parsed entry.
We prefer that the first line of a function's doc references all of
the arguments. In this case, I suggest to reword as follows:
Return value associated with KEY in an ENTRY-DATA.
What ENTRY-DATA is is explained in the very next line"
> +ENTRY-DATA is the data from a parsed password-store entry.
So you don't need to repeat that in the first line.
> --- a/etc/NEWS
> +++ b/etc/NEWS
> @@ -1485,6 +1485,21 @@ the new variable 'buffer-auto-revert-by-notification'
> to a non-nil
> value. Auto Revert mode can use this information to avoid polling the
> buffer periodically when 'auto-revert-avoid-polling' is non-nil.
>
> +** auth-source-pass
> +
> +*** New customizable variable 'auth-source-pass-filename'.
> +Allows setting the path to the password-store, defaults to
> +~/.password-store.
> +
> +*** New customizable variable 'auth-source-pass-port-separator'.
> +Specifies separator between host and port, defaults to colon ":".
> +
> +*** Minimize the number of decryptions during password lookup.
> +This makes the package usable with physical tokens requiring touching
> +a sensor for every decryption.
> +
> +*** 'auth-source-pass-get' is now autoloaded.
Do these changes warrant changes in auth.texi manual?
Thanks for working on this.
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Damien Cassou, 2019/06/02
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Noam Postavsky, 2019/06/06
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Damien Cassou, 2019/06/08
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass,
Eli Zaretskii <=
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Noam Postavsky, 2019/06/08
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Damien Cassou, 2019/06/13
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Noam Postavsky, 2019/06/13
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Damien Cassou, 2019/06/14
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Eli Zaretskii, 2019/06/14
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Damien Cassou, 2019/06/14
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Eli Zaretskii, 2019/06/22
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Damien Cassou, 2019/06/24
- bug#36052: 26.2.50; [PATCH] Improve auth-source-pass, Eli Zaretskii, 2019/06/24