GNU bug report logs - #3957
split window for remote commands through eshell

Previous Next

Package: emacs;

Reported by: Colin Williams <lackita <at> gmail.com>

Date: Tue, 28 Jul 2009 19:10:05 UTC

Severity: normal

Done: Michael Albinus <michael.albinus <at> gmx.de>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Colin Williams <lackita <at> gmail.com>
Cc: 3957 <at> debbugs.gnu.org
Subject: bug#3957: split window for remote commands through eshell
Date: Wed, 29 Jul 2009 11:42:20 -0400
> Basically, any time a remote command gets executed through eshell, if
> the frame is a single window it gets split into two windows.
> Essentially, this is because tramp does its magic through
> shell-command, which pops up a window if the output is too large to
> fit in the minibuffer.

I think the problem is not really in Tramp since Tramp just implements
shell-command according to its docstring; C-h v shell-command RET:

  [...]
  Execute string command in inferior shell; display output, if any.

I.e. the problem is that Eshell shouldn't use shell-command but
something else.
[ And probably shell-command should not be a file operation
  (i.e. shouldn't go through find-file-name-handler), instead it should
  use process-file or start-file-process. ]

> I'm currently trying to figure out how to prevent the root split, so
> things like new frames are blocked as well.

I think the right fix might be to make eshell-external-command (and
eshell-remote-command) use process-file or start-file-process.

Tho for async commands (that end in &), shell-command's behavior to open
a separate window/buffer for the command's output probably makes a lot
of sense.  So maybe this bug-report is only for remote commands that
don't end in &.


        Stefan



This bug report was last modified 11 years and 117 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.