From: Glenn Morris <rgm@gnu.org>
To: "Ota\, Takaaki" <Takaaki.Ota@am.sony.com>
Cc: emacs-devel@gnu.org
Subject: Re: comint-password-prompt-regexp in comint.el
Date: Fri, 17 May 2013 15:02:51 -0400 [thread overview]
Message-ID: <327gixwht0.fsf@fencepost.gnu.org> (raw)
In-Reply-To: <20130516.150150.226695145.Takaaki.Ota@am.sony.com> (Takaaki Ota's message of "Thu, 16 May 2013 15:01:50 -0700")
"Ota, Takaaki" wrote:
> Would you consider the attached modification to cover the next case?
[...]
> Password for 'https://takota@bitbucket.org':
[...]
> *** comint.el 2013-05-16 14:54:43.632331049 -0700
> --- comint.el.mod 2013-05-16 14:53:37.608329808 -0700
> ***************
> *** 349,355 ****
> (regexp-opt
> '("password" "Password" "passphrase" "Passphrase"
> "pass phrase" "Pass phrase" "Response"))
> ! "\\(?:\\(?:, try\\)? *again\\| (empty for no passphrase)\\| (again)\\)?\
> \\(?: for [^:]+\\)?:\\s *\\'")
> "Regexp matching prompts for passwords in the inferior process.
> This is used by `comint-watch-for-password-prompt'."
> --- 349,355 ----
> (regexp-opt
> '("password" "Password" "passphrase" "Passphrase"
> "pass phrase" "Pass phrase" "Response"))
> ! "\\(?:\\(?:, try\\)? *again\\| (empty for no passphrase)\\| (again)\\| for '.*'\\)?\
> \\(?: for [^:]+\\)?:\\s *\\'")
> "Regexp matching prompts for passwords in the inferior process.
> This is used by `comint-watch-for-password-prompt'."
Wouldn't something simpler be better?
--- lisp/comint.el 2013-05-15 23:55:41 +0000
+++ lisp/comint.el 2013-05-17 19:01:22 +0000
@@ -351,7 +351,7 @@
'("password" "Password" "passphrase" "Passphrase"
"pass phrase" "Pass phrase" "Response"))
"\\(?:\\(?:, try\\)? *again\\| (empty for no passphrase)\\|
(again)\\)?\
-\\(?: for [^:]+\\)?:\\s *\\'")
+\\(?: for .+\\)?:\\s *\\'")
"Regexp matching prompts for passwords in the inferior process.
This is used by `comint-watch-for-password-prompt'."
:version "24.1"
PS I think it would have been better to report this as a bug.
next prev parent reply other threads:[~2013-05-17 19:02 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-16 22:01 comint-password-prompt-regexp in comint.el Ota, Takaaki
2013-05-17 19:02 ` Glenn Morris [this message]
2013-05-17 20:29 ` Ota, Takaaki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=327gixwht0.fsf@fencepost.gnu.org \
--to=rgm@gnu.org \
--cc=Takaaki.Ota@am.sony.com \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.