Re: problem with tramp and uuencode-decode

[Han Boetes]

Kai Grossjohann wrote:
> Han Boetes <address@hidden> writes:
>
> > Please apply this patch: [using "-o /dev/stdout" instead of
> > "-o -" because of Posix compliance]
>
> I suggest not to replace "-o -" with "-o /dev/stdout". Instead,
> add another entry in the list so that "-o /dev/stdout" is tried
> first, then "-o -".
>
> It is important for Tramp to work with as many remote systems as
> possible, even non-Posix ones. There are users who wish to
> access AIX and IRIX systems, and these are very different from
> what you are accustomed to.

That's sound logic, here is a new patch.



# Han

* looking for address@hidden/emacs--multi-tty--0--patch-263 to compare with
* comparing to address@hidden/emacs--multi-tty--0--patch-263
M  lisp/net/tramp.el
* modified files

--- orig/lisp/net/tramp.el
+++ mod/lisp/net/tramp.el
@@ -1535,8 +1535,9 @@
 rm -f /tmp/tramp.$$
 }"
   "Shell function to implement `uudecode' to standard output.
-Many systems support `uudecode -o -' for this or `uudecode -p', but
-some systems don't, and for them we have this shell function.")
+Many systems support `uudecode -o /dev/stdout,' `uudecode -p' or
+`uudecode -o -' for this, but some systems don't, and for them we
+have this shell function.")
 
 ;; Perl script to implement `file-attributes' in a Lisp `read'able
 ;; output.  If you are hacking on this, note that you get *no* output
@@ -5970,10 +5971,12 @@
      base64-encode-region base64-decode-region)
     ("recode data..base64" "recode base64..data"
      base64-encode-region base64-decode-region)
-    ("uuencode xxx" "uudecode -o -"
+    ("uuencode xxx" "uudecode -o /dev/stdout"
      tramp-uuencode-region uudecode-decode-region)
     ("uuencode xxx" "uudecode -p"
      tramp-uuencode-region uudecode-decode-region)
+    ("uuencode xxx" "uudecode -o -"
+     tramp-uuencode-region uudecode-decode-region)
     ("uuencode xxx" "tramp_uudecode"
      tramp-uuencode-region uudecode-decode-region)
     ("tramp_encode_with_module" "tramp_decode_with_module"