bug#65795: closed (30.0.50; [PATCH] Keep track of all processes in Eshell pipelines)

[GNU bug Tracking System]

Your message dated Sun, 10 Sep 2023 10:42:58 -0700
with message-id <491abf7d-2c3f-f0e9-07c5-bcfbf39d0141@gmail.com>
and subject line Re: bug#65795: 30.0.50; [PATCH] Keep track of all processes in 
Eshell pipelines
has caused the debbugs.gnu.org bug report #65795,
regarding 30.0.50; [PATCH] Keep track of all processes in Eshell pipelines
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
65795: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=65795
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: 30.0.50; [PATCH] Keep track of all processes in Eshell pipelines Date: Wed, 6 Sep 2023 21:26:19 -0700 Currently, Eshell only keeps track of the head and tail processes in a pipeline. That works for most things, but it can cause issues with Eshell thinking the pipeline is done too early. If the tail process ends first, we assume the entire pipeline is done, when that's not always true.

These patches change things so that Eshell now keeps track of every process in a pipeline and waits for them to be done before proceeding on with the rest of Eshell's evaluation. That's less error-prone, and matches the behavior of other shells.

(This also helps prepare the codebase for the addition of a new feature for Eshell: job control. I have a mostly-working version of that locally, so once this merges and I've finished the job control patches, I'll post them to a new bug.)

0001-Move-common-Eshell-pipeline-code-to-a-separate-funct.patch
Description: Text document

0002-Make-Eshell-synchronous-pipeline-code-more-similar-t.patch
Description: Text document

0003-Collect-all-processes-in-an-Eshell-pipeline-not-just.patch
Description: Text document

0004-Move-some-Eshell-tests-to-more-topical-files.patch
Description: Text document

0005-Wait-for-all-processes-in-a-pipeline-before-resuming.patch
Description: Text document

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#65795: 30.0.50; [PATCH] Keep track of all processes in Eshell pipelines
>
>
>
>
> Date: 
>
> Sun, 10 Sep 2023 10:42:58 -0700
>
>
>
>
> Version: 30.1
>
> On 9/6/2023 9:26 PM, Jim Porter wrote:
> > These patches change things so that Eshell now keeps track of every 
> > process in a pipeline and waits for them to be done before proceeding on 
> > with the rest of Eshell's evaluation. That's less error-prone, and 
> > matches the behavior of other shells.
>
> Merged to master as 2ec41c174f9. Closing this now.
>
>
> --- End Message ---