diff options
| author | Tommaso Negri <me@tommasongr.com> | 2026-10-15 11:01:01 +0200 |
|---|---|---|
| committer | Tommaso Negri <me@tommasongr.com> | 2026-10-15 11:01:01 +0200 |
| commit | 758f757cb9b60dddf10efd5d37becfec9f4b5196 (patch) | |
| tree | 8c9a6ceea48a5981765e2150ea01890544bd231e /tasks/20261014-191209 | |
| parent | 498ac51307412adc29d6d4a6240ae7b98d55621e (diff) | |
| download | bar-758f757cb9b60dddf10efd5d37becfec9f4b5196.tar.gz bar-758f757cb9b60dddf10efd5d37becfec9f4b5196.zip | |
drain command stdout and stderr concurrently
Diffstat (limited to 'tasks/20261014-191209')
| -rw-r--r-- | tasks/20261014-191209/TASK.md | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tasks/20261014-191209/TASK.md b/tasks/20261014-191209/TASK.md index 2509308..bd127bc 100644 --- a/tasks/20261014-191209/TASK.md +++ b/tasks/20261014-191209/TASK.md @@ -13,3 +13,9 @@ Approach: First reproduce a SIGTERM-ignoring command and a shell with a descenda Completion checks: Normal commands complete without later signals. Both reproductions return ERROR within timeout plus documented grace, release readers, and leave no owned child running. Allow a short timeout in tests without adding a user-facing configuration system. Review together with concurrent pipe draining. Design constraint: Follow README.md: keep source-based configuration, no dependencies or plugin framework, and use the smallest readable change that makes the existing behavior reliable. This is a suggestion for individual review, not a deadline. Findings were recorded on 2026-10-14; re-check the current code before implementing. + +Integration note (2026-10-15): task 20261014-190252 now drains stderr on one GCD +worker while the caller drains stdout, and joins the worker with a DispatchGroup. +Runner still owns the single termination timer and cancels it after both readers +and process waiting finish. Bounded cleanup must release both readers; the +concurrent-draining change does not alter the existing timeout guarantees. |