Chris Zheng <[email protected]> writes: > Hello, Michael.
Hi Chris, > With Tramp in master branch of Emacs, I can't open file through > "/sshx:" under MS-Windows. The recipe starting from "emacs -Q" looks > like: > > C-x C-f /sshx:[email protected]#xxxxx:/root/htpasswd.py RET It took me a while to reproduce. The usual test cases don't show up this error. > The related messages look like: > > Tramp: Inserting ‘/sshx:[email protected]#xxxxx:/root/htpasswd.py’... > Tramp: Encoding remote file > ‘/sshx:[email protected]#xxxxx:/root/htpasswd.py’ with ‘base64 <%s’...done > Tramp: Decoding local file ‘z:/TEMP/tramp.16808GLL.py’ with > ‘base64-decode-region’...done > Tramp: Inserting ‘/sshx:[email protected]#xxxxx:/root/htpasswd.py’...failed > Removing old name: No such file or directory, /TEMP/tramp.16808GLL.py Well, you have your temporary directory at z:/TEMP. Per default (at least in the environment I could use for test), it is C:/Temp or C:/Users/USER/AppData/Local/Temp or so. Since "/Temp" is the same directory as "C:/Temp" in Emacs, the problem didn't appear to the tests. > I think the problem is due to the commit > > c5f466db6f6b8196b8429db8bb7b74f8090d5d3e > Fix Bug#20821 > > * lisp/net/tramp.el (tramp-file-name-handler): > * lisp/net/tramp-sh.el (tramp-sh-handle-expand-file-name): > Use `tramp-drop-volume-letter'. (Bug#20821) > > Since the volume letter ("z:") is dropped, the path becomes wrong. I > have read the original report of Bug#20821, but have no idea why the > `tramp-drop-volume-letter' was used here. Comment out the first call > to `tramp-drop-volume-letter' in `tramp-sh-handle-expand-file-name' > did fix the problem for me. Could you please have a look on it? I've checked the whole story of that bug. The first change in `tramp-file-name-handler' is still needed for fixing the bug. But as you've said, calling `tramp-drop-volume-letter' in `tramp-sh-handle-expand-file-name' is not needed anymore. I don't recall why I have applied it that time; maybe it was also needed, and it is now superfluous due to other changes in Tramp. Anyway, I have removed that call as proposed by you, committed to the repositories. Could you, pls, check? > With many thanks, > > Chris Best regards, Michael. _______________________________________________ Tramp-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/tramp-devel
