GNU bug report logs - #65795
30.0.50; [PATCH] Keep track of all processes in Eshell pipelines

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Jim Porter <jporterbugs <at> gmail.com>
Subject: bug#65795: closed (Re: bug#65795: 30.0.50; [PATCH] Keep track of
 all processes in Eshell pipelines)
Date: Sun, 10 Sep 2023 17:44:02 +0000

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

Your bug report

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

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 65795 <at> debbugs.gnu.org.

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

[Message part 2 (message/rfc822, inline)]

From: Jim Porter <jporterbugs <at> gmail.com>
To: 65795-done <at> debbugs.gnu.org
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.

[Message part 3 (message/rfc822, inline)]

From: Jim Porter <jporterbugs <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.0.50; [PATCH] Keep track of all processes in Eshell pipelines
Date: Wed, 6 Sep 2023 21:26:19 -0700

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

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 (text/plain, attachment)]

[0002-Make-Eshell-synchronous-pipeline-code-more-similar-t.patch (text/plain, attachment)]

[0003-Collect-all-processes-in-an-Eshell-pipeline-not-just.patch (text/plain, attachment)]

[0004-Move-some-Eshell-tests-to-more-topical-files.patch (text/plain, attachment)]

[0005-Wait-for-all-processes-in-a-pipeline-before-resuming.patch (text/plain, attachment)]

Previous Next

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