Re: Tramp patch for emacs-24 branch

[Michael Albinus]

Eli Zaretskii <address@hidden> writes:

Hi Eli,

>> Last days, I have worked on a fix for a nasty Tramp problem: it loads
>> itself recursively, when default-directory is already a remote file
>> name.
>
> And why is that a problem?  Does the second load cause any trouble?

It's not a second load, but an infinite recursive load:

emacs -Q -batch --eval '(let ((default-directory "/ssh::")) (expand-file-name 
"foo"))'
Recursive load: "/usr/local/share/emacs/24.3.50/lisp/net/tramp.elc", 
"/usr/local/share/emacs/24.3.50/lisp/net/tramp.elc", 
"/usr/local/share/emacs/24.3.50/lisp/net/tramp.elc", 
"/usr/local/share/emacs/24.3.50/lisp/net/tramp.elc", 
"/usr/local/share/emacs/24.3.50/lisp/net/tramp.elc"

> I'd say it depends on how grave the problem is, see my questions above.

It's a bug. But I believe the corresponding use case does not happen
often.

Best regards, Michael.