[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] externals/multishell 8026840 10/23: Change variable name to respe
From: |
Stefan Monnier |
Subject: |
[elpa] externals/multishell 8026840 10/23: Change variable name to respect gnu conventions. |
Date: |
Sun, 29 Nov 2020 23:19:37 -0500 (EST) |
branch: externals/multishell
commit 80268402f2901d4f5c3275c889892a719ed49083
Author: Ken Manheimer <ken.manheimer@gmail.com>
Commit: Ken Manheimer <ken.manheimer@gmail.com>
Change variable name to respect gnu conventions.
From gnu people's FIXME suggestion.
---
multishell.el | 31 +++++++++++++++++--------------
1 file changed, 17 insertions(+), 14 deletions(-)
diff --git a/multishell.el b/multishell.el
index 3428638..de72b3c 100644
--- a/multishell.el
+++ b/multishell.el
@@ -696,19 +696,22 @@ and path nil if none is resolved."
(declare-function tramp-dissect-file-name "tramp")
(declare-function tramp-cleanup-connection "tramp")
-(defun multishell-start-shell-in-buffer (path)
- "Start, restart, or continue a shell in BUFFER-NAME on PATH.
+(defun multishell-start-shell-in-buffer (where)
+ "Start, restart, or continue a shell in current-buffer on WHERE.
-If the path is remote and includes a directory, cd to that directory on the
remote path.
+If WHERE is remote and includes a directory, cd to that directory on the
+remote host.
If cd fails to an included remote directory, try again without the cd."
(let* ((is-active (comint-check-proc (current-buffer))))
- (when (and path (not is-active))
+ (when (and where (not is-active))
- (when (and (derived-mode-p 'shell-mode) (file-remote-p path))
+ ;; FIXME: file-remote-p does not imply Tramp.
+ ;; Why do we need to do something special for Tramp here?
+ (when (and (derived-mode-p 'shell-mode) (file-remote-p where))
;; Returning to disconnected remote shell - do some tidying.
- ;; Without this cleanup, occasionally restarting a disconnected
+ ;; FIXME: Without this cleanup, occasionally restarting a disconnected
;; remote session, particularly one that includes sudo, results in
;; an untraceable "Args out of range" error. That never happens if
;; we precedeed connection attempts with this cleanup -
@@ -717,20 +720,20 @@ If cd fails to an included remote directory, try again
without the cd."
(tramp-dissect-file-name default-directory 'noexpand)
'keep-debug 'keep-password))
- (if (not (file-remote-p path))
- (cd path)
- (message "Connecting to %s" path)
+ (if (not (file-remote-p where))
+ (cd where)
+ (message "Connecting to %s" where)
(condition-case err
- (cd path)
+ (cd where)
;; "cd: No such directory found via CDPATH environment variable"
(error
(if (string=
(cadr err)
"No such directory found via CDPATH environment variable")
- ;; Try again without dir part of remote path:
- (let* ((final-colon-at (string-match ":[^:]+$" path))
- (sans-dir-path (substring path 0 (1+ final-colon-at)))
- (dir-path (substring path (1+ final-colon-at))))
+ ;; Try again without dir part of remote where:
+ (let* ((final-colon-at (string-match ":[^:]+$" where))
+ (sans-dir-path (substring where 0 (1+ final-colon-at)))
+ (dir-path (substring where (1+ final-colon-at))))
(message "Failed to cd to %s, trying again without..."
dir-path)
(sit-for .5)
- [elpa] externals/multishell bf8c445 05/23: Use appropriate interactive mode., (continued)
- [elpa] externals/multishell bf8c445 05/23: Use appropriate interactive mode., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell ee12b22 08/23: Unravel conditional tangle (and use bound-and-true-p), Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 6b0bc33 12/23: Rectify code formatting, per gnu people's guidance., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell aae3f1e 06/23: Use dolist instead of mapcar., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 5adbb49 09/23: Remove unused variable., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 2b4d9d7 15/23: Rectify code formatting, per gnu people's guidance., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 0d349ee 16/23: Revise in-file change log acknowledging other's work and prior unreleased, Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 9d23005 17/23: Refine multishell-pop-to-shell docstring., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell e26492d 18/23: Refine multishell-pop-to-shell docstring a bit more., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 84a7a57 19/23: Minor wording in acknowledgment., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 8026840 10/23: Change variable name to respect gnu conventions.,
Stefan Monnier <=
- [elpa] externals/multishell 68493fa 11/23: Include lexical binding and adjust header copyright date., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 4fee0bd 07/23: Avoid circular load by depending on load from multishell.el., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 324b382 13/23: Change function name to respect gnu convention., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 452e170 14/23: Use changed function name., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 19901e0 20/23: Mark for autoloading some multishell functions used by multishell-list., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell a1d803b 21/23: Make header consistent with elpa copy., Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 1f77d77 22/23: * packages/multishell: require multishell from multishell-list,, Stefan Monnier, 2020/11/29
- [elpa] externals/multishell 35a0dd3 23/23: * multishell/multishell.el: Require cl-lib when compiling for cl-letf macro., Stefan Monnier, 2020/11/29