GNU bug report logs -
#60083
29.0.60; eshell-elecslash when in the root directory of a remote host
Previous Next
Reported by: montag451 <montag451 <at> laposte.net>
Date: Thu, 15 Dec 2022 07:44:02 UTC
Severity: normal
Found in version 29.0.60
Done: Jim Porter <jporterbugs <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
On 12/18/2022 2:34 AM, Eli Zaretskii wrote:
>> Date: Thu, 15 Dec 2022 00:46:50 +0100
>> From: montag451 via "Bug reports for GNU Emacs,
>> the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
>>
>> While testing the new eshell module 'eshell-elecslash' I found a bug
>> when the default-directory is the root of a remote host. To reproduce
>> the problem type the following in a eshell buffer (replacing "method"
>> and "host" by appropriate values):
>>
>> $ cd /method:host:/
>> $ cd / <--- here the "/" should be replaced by "/method:host:/" but
>> instead nothing happen
>>
>> I've attached a patch to this email which fixes this issue.
>
> Jim, are you looking into this issue?
Thanks. I missed the original message. The patch looks good to me,
although it compiles with the following warning:
In eshell-electric-forward-slash:
../../lisp/eshell/em-elecslash.el:98:18: Warning: Unused lexical
variable `localname'
Looking through the code, I think we can just remove that variable.
Since this is a small change, and fixes a corner case in a new feature,
I think it would make sense to go on the release branch, though I'll
defer to you on this, Eli. The fact that the new code (with the removal
of 'localname') is a bit simpler helps too.
This bug report was last modified 2 years and 169 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.