Re: git: a85d870007e7 - main - man.sh: revert trapping SIGPIPE

From: Herbert J. Skuhra <herbert_at_gojira.at>
Date: Mon, 11 Nov 2024 14:10:30 UTC
On Fri, 28 Jun 2024 02:35:20 +0200, Warner Losh wrote:
> 
> The branch main has been updated by imp:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=a85d870007e7dd94242a2084dbb750438332ac1e
> 
> commit a85d870007e7dd94242a2084dbb750438332ac1e
> Author:     Alexander Ziaee <concussious@runbox.com>
> AuthorDate: 2024-06-10 04:04:51 +0000
> Commit:     Warner Losh <imp@FreeBSD.org>
> CommitDate: 2024-06-28 00:34:33 +0000
> 
>     man.sh: revert trapping SIGPIPE
>     
>     PR:             279542
>     Fixes:          14a5c1068d37
>     Reported by:    emaste
>     Reviewed by: imp, emaste, jilles
>     Pull Request: https://github.com/freebsd/freebsd-src/pull/1283
> ---
>  usr.bin/man/man.sh | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/usr.bin/man/man.sh b/usr.bin/man/man.sh
> index 7461153942f5..f4037ed92215 100755
> --- a/usr.bin/man/man.sh
> +++ b/usr.bin/man/man.sh
> @@ -36,9 +36,6 @@ ulimit -t 20
>  # do not ignore the exit status of roff tools
>  set -o pipefail
>  
> -# ignore SIGPIPE exits because pagers may exit before reading all their input.
> -trap '' SIGPIPE
> -
>  # Usage: add_to_manpath path
>  # Adds a variable to manpath while ensuring we don't have duplicates.
>  # Returns true if we were able to add something. False otherwise.

Should this be merged to stable/14 and releng/14.2?