From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Michael Albinus Newsgroups: gmane.emacs.devel Subject: Re: Recommend move eshell/su and eshell/sudo to em-tramp.el Date: Tue, 03 Jul 2012 08:56:17 +0200 Message-ID: <87pq8dl4dq.fsf@gmx.de> References: NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1341298591 16340 80.91.229.3 (3 Jul 2012 06:56:31 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Tue, 3 Jul 2012 06:56:31 +0000 (UTC) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Jul 03 08:56:31 2012 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Slx2C-00023v-KE for ged-emacs-devel@m.gmane.org; Tue, 03 Jul 2012 08:56:28 +0200 Original-Received: from localhost ([::1]:55504 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Slx2B-0007g4-JX for ged-emacs-devel@m.gmane.org; Tue, 03 Jul 2012 02:56:27 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:50113) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Slx28-0007fw-GM for emacs-devel@gnu.org; Tue, 03 Jul 2012 02:56:25 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Slx26-0001mM-PG for emacs-devel@gnu.org; Tue, 03 Jul 2012 02:56:24 -0400 Original-Received: from mailout-de.gmx.net ([213.165.64.22]:58614) by eggs.gnu.org with smtp (Exim 4.71) (envelope-from ) id 1Slx26-0001m2-Fc for emacs-devel@gnu.org; Tue, 03 Jul 2012 02:56:22 -0400 Original-Received: (qmail invoked by alias); 03 Jul 2012 06:56:19 -0000 Original-Received: from p57BB976D.dip0.t-ipconnect.de (EHLO detlef.gmx.de) [87.187.151.109] by mail.gmx.net (mp072) with SMTP; 03 Jul 2012 08:56:19 +0200 X-Authenticated: #3708877 X-Provags-ID: V01U2FsdGVkX1+gQ7oyzkag17oPx4qV8uJ4QIA8ZJsG+wL/E325ny dvalOiENdmLPHB In-Reply-To: (John Wiegley's message of "Mon, 02 Jul 2012 14:25:11 -0500") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.1.50 (gnu/linux) X-Y-GMX-Trusted: 0 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 213.165.64.22 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:151385 Archived-At: John Wiegley 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.