GNU bug report logs - #71554
29.3; eshell-command async buffer behavior

Previous Next

Package: emacs;

Reported by: Christopher Howard <christopher <at> librehacker.com>

Date: Fri, 14 Jun 2024 13:58:01 UTC

Severity: normal

Found in version 29.3

Done: Jim Porter <jporterbugs <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


Message #26 received at 71554 <at> debbugs.gnu.org (full text, mbox):

From: Jim Porter <jporterbugs <at> gmail.com>
To: Thierry Volpiatto <thievol <at> posteo.net>
Cc: 71554 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
 christopher <at> librehacker.com
Subject: Re: bug#71554: 29.3; eshell-command async buffer behavior
Date: Sun, 23 Jun 2024 22:36:47 -0700
On 6/20/2024 12:30 AM, Thierry Volpiatto wrote:
> This patch doesn't work if user kill for some reason the initial process
> buffer, we have to check if other buffers are alive.  Also having a new
> variable eshell-command-async-buffer instead of reusing
> async-shell-command-buffer is better IMO.
> Here a patch that fix these issues.

I haven't tried it yet, but this patch makes sense to me, with the tiny 
exception that I think defaulting to 'confirm-new-buffer' would be 
better for consistency. (It's not my preferred setting, but I think we 
should stay the same as Comint unless we have a good reason for why that 
value has some problems unique to Eshell.)

Eli, what do you think about merging this into the Emacs 30 branch? I 
have no strong preference myself, but I think it's small enough that it 
should be safe, and it fixes an annoyance with Eshell today. (I can also 
write a regression test or two if that would help matters.)




This bug report was last modified 318 days ago.

Previous Next


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