patch: wrong behaviour of shell-command handler

[Ivan Zakharyaschev]

        Hello!

I'm sorry for reposting the message: I haven't noticed the list address
the first time.

---------- Forwarded message ----------

I've just run across a simple bug (at least, it looks to me like a bug)
in tramp.el from tramp-2.0.22 (and from 2.0.24 as well).

It should be obvious from the patch what I mean.

Here is a longer description in words of the fix:

The three part of tramp-handle-shell-command function's body are
evaluated one after another, and so the final strange fallback
expressions get evaluated in normal situations: a warning about wrong
default-directory appears, and the command returns errors. Putiing them
into an (if ) expression as the else-case fixes the bug.


By the way, are there any plans on implementing asynchronous
shell-command execution: do you expect it to be hard to do? This feature
is missing me, so perhaps I could take part in implementing it, if I had
some guidelines on how to do that. (I'm trying to setup Emacs+TRAMP for
editing remote RPM spec files and compiling them remotely on the build
server;  I almost like the way it works at the moment, but I cannot
observe the output of rpmbuild as it appears, only after it completes;
that's why I'd like to have the asynchronous execution feature
implemented.)

Regards,

-- 
Ivan Zakharyaschev
ALT Linux Team member, Sisyphus developer
http://www.altlinux.ru, http://www.altlinux.com

emacs-tramp-2.0.22-alt1-shell-not-local.patch
Description: sorry, relative to my /