[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Two Tramp issues

From: Kai Großjohann
Subject: Re: Two Tramp issues
Date: Sat, 03 Aug 2002 10:51:19 +0200
User-agent: Gnus/5.090007 (Oort Gnus v0.07) Emacs/21.3.50 (i686-pc-linux-gnu)

"Lars Hansen" <address@hidden> writes:

> 1. Bug:
>    It seems that current buffer is changed to be the Tramp buffer
>    when Tramp handles `make-directory'.

Right.  Please try this patch.

--- tramp.el.~2.162.~   Wed Jul 31 17:23:43 2002
+++ tramp.el    Sat Aug  3 10:48:04 2002
@@ -2407,13 +2407,14 @@
   (with-parsed-tramp-file-name dir nil
     (when (tramp-ange-ftp-file-name-p multi-method method)
       (tramp-invoke-ange-ftp 'make-directory dir parents))
-    (tramp-barf-unless-okay
-     multi-method method user host
-     (format " %s %s"
-            (if parents "mkdir -p" "mkdir")
-            (tramp-shell-quote-argument path))
-     nil 'file-error
-     "Couldn't make directory %s" dir)))
+    (save-excursion
+      (tramp-barf-unless-okay
+       multi-method method user host
+       (format " %s %s"
+              (if parents "mkdir -p" "mkdir")
+              (tramp-shell-quote-argument path))
+       nil 'file-error
+       "Couldn't make directory %s" dir))))
 ;; CCC error checking?
 (defun tramp-handle-delete-directory (directory)

> 2. Question:
>    Is it correct that previosly a function such as eg. `make-directory'
>    was not caught by the "file name magic" when `make-directory' was
>    called with a relative file name and default-directory was a Tramp
>    file name -- but that this is now corrected ?
>    It so, is this change documented ?

Well, this is possible.  I recently did this change:

2002-07-26  Kai Großjohann  <address@hidden>

        * tramp.el (tramp-coding-commands): New variable.
        (tramp-find-inline-encoding): New function.
        (tramp-handle-make-directory, tramp-handle-delete-directory)
        (tramp-handle-delete-file): Do expand-file-name on arg.

This is from the ChangeLog in Tramp.  Maybe this detail got lost in
the ChangeLog in Emacs.

A large number of young women don't trust men with beards.  (BFBS Radio)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]