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

Previous Next

Full log

View this message in rfc822 format

From: Thierry Volpiatto <thievol <at> posteo.net>
To: Jim Porter <jporterbugs <at> gmail.com>
Cc: Thierry Volpiatto <thievol <at> posteo.net>, 71554 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>, christopher <at> librehacker.com
Subject: bug#71554: 29.3; eshell-command async buffer behavior
Date: Mon, 24 Jun 2024 06:23:10 +0000

[Message part 1 (text/plain, inline)]

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

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

Of course, I just forget setting the default back to
'confirm-new-buffer' in the patch.

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

-- 
Thierry

[signature.asc (application/pgp-signature, inline)]

Previous Next

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