Mark Geary <ge...@acm.org> writes:

Hi Mark,

> I tried to apply the patch, then realized I needed to update Tramp to 2.4.2
> (I had 2.4.1).

Yes, I prepare patches in the git repository.

> I also realized that I needed to delete ~/emacs.d/tramp.
> (Which, I suppose, is why Tramp had been using /usr/ccs/bin/bash.)

Maybe this led to problems. I see now the errors

--8<---------------cut here---------------start------------->8---
> 16:57:08.574453 tramp-file-name-handler (1) # Symbol’s value as variable is 
> void: "Symbol's value as variable is void", 
> "(tramp-inline-compress-start-size)"
> 16:57:08.574572 tramp-file-name-handler (1) # Symbol’s value as variable is 
> void: "Symbol's value as variable is void", "(tramp-copy-size-limit)"
> 16:57:08.574640 tramp-file-name-handler (1) # Symbol’s value as variable is 
> void: "Symbol's value as variable is void", "(tramp-histfile-override)"
> 16:57:08.574875 tramp-file-name-handler (1) # Symbol’s value as variable is 
> void: "Symbol's value as variable is void", 
> "(tramp-use-ssh-controlmaster-options)"
--8<---------------cut here---------------end--------------->8---

Instead of removing ~/emacs.d/tramp, you might call "M-x 
tramp-cleanup-all-connections".
This is safer.

> I applied the patch and re-installed Tramp.
> It still fails when I try to edit a file on a remote Solaris 11 box.
> (Though Emacs no longer hangs at the failure.)
>
>     Tramp: Inserting ‘/ssh:sfmg10:/home/mag43/.profile’...failed
>     (Wrong method specification for ‘ssh’)
>
> I've attached the debug file.

Thanks. I see now the error

--8<---------------cut here---------------start------------->8---
> 16:57:15.722371 tramp-barf-unless-okay (1) # File error: ‘( (test -e 
> /home/mag43/.profile || test -h /home/mag43/.profile) && (env 
> QUOTING_STYLE=locale /usr/bin/stat -c '((/////%N/////) %h %u %g %X %Y %Z %s 
> /////%A///// t %i -1)' /home/mag43/.profile | sed -e 's/"/\\"/g' -e 
> 's/\/\/\/\/\//"/g') || echo nil)’ returns with error
> bash: null: command not found
--8<---------------cut here---------------end--------------->8---

The command is shorter than 250 chars, so it must be something
different. "stat" is still involved, I start to believe we shall simply
ban stat for remote Solaris.

I`ve prepared another patch, which does it. Pls remove the first patch,
and install this instead. And run "M-x tramp-cleanup-all-connections"
before testing.

> Mark

Best regards, Michael.

diff --git a/lisp/tramp-sh.el b/lisp/tramp-sh.el
index b7089207..08f05630 100644
--- a/lisp/tramp-sh.el
+++ b/lisp/tramp-sh.el
@@ -4207,6 +4207,13 @@ process to set up.  VEC specifies the connection."
       (tramp-cleanup-connection vec t t)
       (throw 'uname-changed (tramp-maybe-open-connection vec)))

+    ;; stat on Solaris is buggy.  We've got reports for "SunOS 5.10"
+    ;; and "SunOS 5.11" so far.
+    (when (string-match-p
+          (eval-when-compile (regexp-opt '("SunOS 5.10" "SunOS 5.11")))
+          (tramp-get-connection-property vec "uname" ""))
+      (tramp-set-connection-property vec "stat" nil))
+
     ;; Try to set up the coding system correctly.
     ;; CCC this can't be the right way to do it.  Hm.
     (tramp-message vec 5 "Determining coding system")
_______________________________________________
Tramp-devel mailing list
Tramp-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/tramp-devel

Reply via email to