unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Michael Albinus <michael.albinus@gmx.de>
To: emacs-devel@gnu.org
Subject: Re: Recommend move eshell/su and eshell/sudo to em-tramp.el
Date: Tue, 03 Jul 2012 08:56:17 +0200	[thread overview]
Message-ID: <87pq8dl4dq.fsf@gmx.de> (raw)
In-Reply-To: <m2pq8ec6eg.fsf@newartisans.com> (John Wiegley's message of "Mon,  02 Jul 2012 14:25:11 -0500")

John Wiegley <johnw@newartisans.com> writes:

Hi John,

looks like it is the same message you have sent some days ago. Are there
changes I haven't seen? Anyway ...

> 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)

In a first reaction I've said this doesn't happen if default-directory
is local. This is wrong, I did oversee that I had defined an alias as
metioned in (info "(eshell) Built-ins")

alias sudo '*sudo $*'

Would such an alias serve you better?

>     - It doesn't show any output until the command is done

I'm working on this. Hopefully, I could change the behaviour when the
command is an external command. If the command is mapped to a lisp
function (like the command "ls" to "eshell/ls"), I have no clue how to
achieve this. The latter case happens also for Lisp code not using
Tramp, but there are shorter delays.

>     - It locks up Emacs until the command is done

If the command runs lisp code, you could interrupt with ^G as usual. For
external commands it is more complicate, because I don't know how to
propagate signals to such remote processes in Tramp. The only idea I
have is to interrupt the Tramp connection itself, which sounds like a
sledge-hammer approach.

> I do not believe these 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.

Would it be sufficient to propagate the above mentioned alias more
prominently? Or shall we make that alias the default?

> John

Best regards, Michael.



  reply	other threads:[~2012-07-03  6:56 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-02 19:25 Recommend move eshell/su and eshell/sudo to em-tramp.el John Wiegley
2012-07-03  6:56 ` Michael Albinus [this message]
2012-07-03 19:47   ` John Wiegley
2012-07-04 14:53     ` Michael Albinus
2012-07-10 12:22   ` Michael Albinus
2012-07-10 16:57     ` John Wiegley
2012-07-10 18:56       ` Michael Albinus
2012-07-11  3:57         ` John Wiegley
  -- strict thread matches above, loose matches on Subject: below --
2012-06-29  3:32 John Wiegley
2012-06-29  9:45 ` Michael Albinus
2012-06-29 21:51 ` Phil Hagelberg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87pq8dl4dq.fsf@gmx.de \
    --to=michael.albinus@gmx.de \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).