Two Tramp issues

Two Tramp issues

[Lars Hansen]

Hi

1. Bug:
   It seems that current buffer is changed to be the Tramp buffer
   when Tramp handles `make-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 ?

I use Emacs 20 and 21 and `tramp-version' is
"$Id: tramp.el,v 2.95 2002/04/10 17:16:25 kaig Exp $"

Lars

Re: Two Tramp issues

[Kai Großjohann]

"Lars Hansen" <larsh@math.ku.dk> 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  <Kai.Grossjohann@CS.Uni-Dortmund.DE>

	* 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.

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