opensubscriber
   Find in this group all groups
 
Unknown more information…

e : emacs-devel@gnu.org 29 June 2012 • 11:32AM -0400

Recommend move eshell/su and eshell/sudo to em-tramp.el
by John Wiegley

REPLY TO AUTHOR
 
REPLY TO GROUP




Something I've noticed after moving to Emacs 24 is that using "sudo" in Eshell
has become an awful experience:

    - It's *much* slower than *sudo (by orders of magnitude)
    - It doesn't show any output until the command is done
    - It locks up Emacs until the command is done

I find this behavior distasteful, since sudo is such a common command for
users of Eshell to use.  It was added in 2009 by the Tramp author:

    2009-11-27  Michael Albinus  <michael.albinus@gmx....>

           * eshell/em-unix.el (eshell/su, eshell/sudo): New defuns,
           providing a Tramp related implementation of "su" and "sudo".
           (eshell-unix-initialize): Add "su" and "sudo".

I appreciate what these two functions are trying to accomplish, but they
should not have been made the default.  I recommend moving them out of
em-unix.el and into a new module, em-tramp.el, which is OFF by default.  Users
who prefer to use Tramp for everything can turn it on, and a note to that
effect should be added to the Tramp manual.

John


Bookmark with:

Delicious   Digg   reddit   Facebook   StumbleUpon

Related Messages

opensubscriber is not affiliated with the authors of this message nor responsible for its content.