uucore: centralize SIGPIPE handling in main macro

[ChrisDryden]

Instead of adding this code to every utility, hoping to make it centralized and then addressing all of the special cases in the individual utilities. There's definitely a chance that a special case is missed and could be a regression, but this should solve a much larger number of issues for all utilities.

Fixes #10325
Fixes #10260
Fixes #10230
Fixes #10214
Fixes #9936
Fixes #8919
Fixes #7252
Fixes #4965

[github-actions]

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Note: The gnu test tests/printf/printf-surprise is now being skipped but was previously passing.

[codspeed-hq]

CodSpeed Performance Report

Merging this PR will improve performance by 3.85%

_{Comparing ChrisDryden:sigpipe-main-macro (847fff3) with main (cbbff30)}

Summary

⚡ 1 improved benchmark
✅ 141 untouched benchmarks
⏩ 180 skipped benchmarks¹

Performance Changes

	Mode	Benchmark	BASE	HEAD	Efficiency
⚡	Simulation	sort_ascii_utf8_locale	18.1 ms	17.5 ms	+3.85%

Footnotes

1. 180 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[github-actions]

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)

[Copilot]

Pull request overview

This PR centralizes SIGPIPE handling across all uutils utilities by moving the signal setup logic into the #[uucore::main] procedural macro. Previously, each utility had to manually call enable_pipe_errors() to restore SIGPIPE's default behavior. Now this happens automatically in the macro, with utilities only needing to opt-out when they require special SIGPIPE handling.

Changes:

• Centralized SIGPIPE capture and restoration in the #[uucore::main] proc macro
• Added disable_pipe_errors() function for utilities that need to override default SIGPIPE behavior
• Removed redundant SIGPIPE setup code from 7 utilities (yes, tr, timeout, tail, seq, env, cat)
• Added special SIGPIPE handling for 3 utilities with non-standard requirements (tee, tty, split)

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/uucore_procs/src/lib.rs	Added SIGPIPE state capture at module level and restoration logic before user code executes in the main macro
src/uucore/src/lib/features/signals.rs	Added disable_pipe_errors() function and updated documentation for enable_pipe_errors()
src/uucore/Cargo.toml	Added "signals" feature to default features to enable SIGPIPE handling for all utilities
src/uu/yes/src/yes.rs	Removed redundant enable_pipe_errors() call now handled by macro
src/uu/tty/src/tty.rs	Added disable_pipe_errors() to handle broken pipes gracefully and exit with code 3
src/uu/tr/src/tr.rs	Removed redundant SIGPIPE setup now handled by macro
src/uu/timeout/src/timeout.rs	Removed redundant enable_pipe_errors() call now handled by macro
src/uu/tee/src/tee.rs	Changed logic from enable_pipe_errors() when no output-error to disable_pipe_errors() when output-error is set
src/uu/tail/src/tail.rs	Removed redundant SIGPIPE setup now handled by macro
src/uu/split/src/split.rs	Added conditional disable_pipe_errors() when using --filter option to handle child process stdin closing
src/uu/seq/src/seq.rs	Removed manual SIGPIPE capture and restoration now handled by macro
src/uu/env/src/env.rs	Removed redundant enable_pipe_errors() call now handled by macro
src/uu/cat/src/cat.rs	Removed redundant SIGPIPE setup now handled by macro

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Ecordonnier]

@ChrisDryden this is a good change. I'm OK to merge it. Can you please fix the conflict?

[github-actions]

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)

[ChrisDryden]

Just making a list of all of the PRs and Issues this will close to track for when this closes:
PRs:

• fix(yes): handle trapped SIGPIPE correctly #9560
• yes, timeout: adding pipeline signal handlers #10166
• sort: use uucore SIGPIPE handling for broken pipes  #10264
• feat(install): add SIGPIPE handling for Unix pipeline compatibility #10265

Issues:

• tee: SIGPIPE not ignored for --output-error/-p modes #10325
• sort: piping stdout to head -n 0 gives 'Broken pipe' #10260
• fold: does not handle broken pipe in the same way as GNU #10230
• dd: piping from stdout gives Broken pipe #10214
• all: manyutil --version | tee > dir broke pipe #9936
• SIGPIPE handling is missing in many utilities #8919
• yes: fails to trap SIGPIPE #7252
• tail does not exit when pipe is broken #4965

[Ecordonnier]

I think the best way to do this is to use the "Fixes" syntax, then github automatically closes the issues when the PR is merged.