man: do not ignore the exit status of roff tools

PR:		223516
Approved by:	emaste, bapt
Differential Revision:	https://reviews.freebsd.org/D44798

This commit is contained in:

Wolfram Schneider

2024-04-20 08:24:58 +00:00

parent 21ffadb6e6

commit 14a5c1068d

1 changed files with 7 additions and 1 deletions

									
										usr.bin/man/man.sh
									
		+7
		-1
	
												View File
												
					@@ -33,6 +33,12 @@

					# it is better to terminate it.

					# it is better to terminate it.

					ulimit -t 20

					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

					# Usage: add_to_manpath path

					# Adds a variable to manpath while ensuring we don't have duplicates.

					# Adds a variable to manpath while ensuring we don't have duplicates.

					# Returns true if we were able to add something. False otherwise.

					# Returns true if we were able to add something. False otherwise.

					@@ -312,7 +318,7 @@ man_check_for_so() {

						# We need to loop to accommodate multiple .so directives.

						# We need to loop to accommodate multiple .so directives.

						while true

						while true

						do

						do

							line=$($cattool "$manpage" | head -n1)

							line=$($cattool "$manpage" 2>/dev/null | head -n1)

							case "$line" in

							case "$line" in

							.so*)	trim "${line#.so}"

							.so*)	trim "${line#.so}"

								decho "$manpage includes $tstr"

								decho "$manpage includes $tstr"