Re: master 7f631a3e2ac: Allow using multiple buffers in 'eshell-command'

[Jim Porter <jporterbugs@gmail.com>]

On 7/6/2024 10:15 AM, Stefan Kangas wrote:
> Jim Porter <jporterbugs@gmail.com> writes:
> 
>> branch: master
>> commit 7f631a3e2aca97e95b8659c902c25ab21f084e08
>> Author: Thierry Volpiatto <thievol@posteo.net>
>> Commit: Jim Porter <jporterbugs@gmail.com>
>>
>>      Allow using multiple buffers in 'eshell-command'
[snip]
> This new test is failing here:

Hm, I can't reproduce that, but I'm guessing it's a race condition with 
the process sentinel. Does this make the test pass for you?

----------------------------------------

diff --git a/lisp/eshell/eshell.el b/lisp/eshell/eshell.el
index 568f6745067..6c79f753d6f 100644
--- a/lisp/eshell/eshell.el
+++ b/lisp/eshell/eshell.el
@@ -357,8 +357,8 @@ eshell-command
                (with-current-buffer bufname
                  ;; Stop all the processes in the old buffer (there may
                  ;; be several).
-                (eshell-process-interact #'interrupt-process t))
-              (accept-process-output)
+                (eshell-process-interact #'interrupt-process t)
+                (eshell-wait-for-process (mapcar #'car 
eshell-process-list)))
                (kill-buffer bufname))
               ((eq eshell-command-async-buffer 'confirm-new-buffer)
                (shell-command--same-buffer-confirm "Use a new buffer")