Differential D50302

man: Exit cleanly on SIGPIPE.
ClosedPublic
Actions

Authored by des on May 11 2025, 10:02 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Apr 29, 1:40 AM

	Unknown Object (File)
	Wed, Apr 29, 1:32 AM

	Unknown Object (File)
	Tue, Apr 28, 12:11 PM

	Unknown Object (File)
	Wed, Apr 22, 6:59 PM

	Unknown Object (File)
	Tue, Apr 21, 8:02 PM

	Unknown Object (File)
	Mon, Apr 20, 6:15 AM

	Unknown Object (File)
	Mon, Apr 20, 5:03 AM

	Unknown Object (File)
	Sun, Apr 19, 7:55 PM

View All 88 Files

Subscribers

Details

Reviewers

ziaee
kevans

Group Reviewers

Commits

rG84b971c9186a: man: Exit cleanly on SIGPIPE.
rG2f99190a9234: man: Exit cleanly on SIGPIPE.
rGfbaba7aa4322: man: Exit cleanly on SIGPIPE.

Summary

The first attempt at addressing this simply suppressed SIGPIPE, which
resulted in mandoc printing out error messages instead. This was then
reverted, but the pipefail was (correctly) left in, so man still
returned a nonzero exit code if you quit a page before the end.

Fixes: 14a5c1068d37, a85d870007e7
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.May 11 2025, 10:02 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 11 2025, 10:02 PM

des requested review of this revision.May 11 2025, 10:02 PM

Harbormaster completed remote builds in B64094: Diff 155305.May 11 2025, 10:02 PM

LGTM + tested, thank you so much!

This revision is now accepted and ready to land.May 11 2025, 10:19 PM

kevans accepted this revision.May 16 2025, 1:13 PM

Closed by commit rGfbaba7aa4322: man: Exit cleanly on SIGPIPE. (authored by des). · Explain WhyMay 16 2025, 2:57 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGfbaba7aa4322: man: Exit cleanly on SIGPIPE..

des added a commit: rG2f99190a9234: man: Exit cleanly on SIGPIPE..May 21 2025, 4:57 PM

des added a commit: rG84b971c9186a: man: Exit cleanly on SIGPIPE..May 21 2025, 5:37 PM

Revision Contents
Changeset List

Path

Size

usr.bin/

man/

15 lines

Diff 155562

usr.bin/man/man.sh

Loading...