Eli Zaretskii wrote:
Date: Tue, 19 Dec 2006 01:23:18 +0100
From: Lennart Borgman <[EMAIL PROTECTED]>
Cc: [email protected]
I believe the problem is in comint-dynamic-complete-as-filename. The let
variable dirsuffix has a hardcoded "/". Maybe a fix could look like this
(dirsuffix (cond ((not comint-completion-addsuffix)
"")
((not (consp comint-completion-addsuffix))
(if (w32-shell-dos-semantics)
"\\"
"/"))
but I am not sure at all. Even if this is correct however I think that
shell-file-name and explicit-shell-file-name must be made buffer local
in the interactive shell buffers. (And did I not mention before that
w32-shell-dos-semantics needs to test explicit-shell-file-name too?)
I really don't think we should try to open this can of worms now.
Let's wait until after the release.
As I said this is clearly a bug. Could you please comment on my
tentative solution? What do you think can go wrong?
My solution also includes adding to shell-mode two lines that makes
shell-file-name and explicit-shell-file-name buffer local:
(make-local-variable 'list-buffers-directory)
(make-local-variable 'explicit-shell-file-name)
(make-local-variable 'shell-file-name)
I believe steps similar to those are necessary to a solution. This must
be done in one form or the other. It might not be sufficient and it
might introduce other similar bugs, but that is in my opinion not enough
to avoid them, not even now.
I will add the above to my patched version, as usual.
_______________________________________________
emacs-pretest-bug mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug
