Re: The “//” fix confuses cygwin-mount

[Michael Albinus]

Andy Moreton <address@hidden> writes:

Hi Andy,

>> I don't know cygwin-mount.el. On the marmalade repo, I found it in
>> version "2001". Installed and evall'ed locally, and rerun the test
>
> To get cygwin-mount.el to do anything useful, you must:
> 1) Install Cygwin (e.g. in "c:\cygwin")
> 2) Set `cygwin-mount-cygwin-bin-directory' to "c:\cygwin\bin", or
>    add "c:\cygwin\bin" to `exec-path'
> 3) Eval "(cygwin-mount-activate)
>
> That should setup the file handlers to do the POSIX<->Win32 path translation.

Ahh, indeed. I could reproduce the problem then. The following patch
seems to fix the problem, could you pls test? (The line numbers might
differ a little bit)

diff --git a/lisp/tramp.el b/lisp/tramp.el
index 92d165e4..0d56ec91 100644
--- a/lisp/tramp.el
+++ b/lisp/tramp.el
@@ -3625,7 +3625,11 @@ support symbolic links."
            (setq filename
                  (concat (file-remote-p filename)
                          (replace-regexp-in-string
-                           "\\`/+" "/" (substitute-in-file-name 
localname)))))))
+                           "\\`/+" "/"
+                          ;; We must disable cygwin file name
+                          ;; handlers and alike.
+                          (tramp-run-real-handler
+                           'substitute-in-file-name (list localname))))))))
       ;; "/m:h:~" does not work for completion.  We use "/m:h:~/".
       (if (and (stringp localname) (string-equal "~" localname))
          (concat filename "/")

>     AndyM

Best regards, Michael.