man.c: minor KNF

man: remove unused global variable 'instype' (since yesterday)

No functional change.

man: remove trailing whitespace

No binary change.

man: fix type mismatch between enum and int (since yesterday)

No binary change.

man.c: fix -m option so it works as documented

Refactoring work in man.c r. 1.40 from twelve years ago introduced a
regression where input from the -m option was appended rather than
prepended to the search paths. Problem reported by C. Chapman on
netbsd-users.

Exit successfully after printing the search path, stop further processing.

Continuing to process had the unintended effect that `man` failed to find
a matching manual page for {EMPTY LIST OF REQUESTED MANUAL PAGES}, and
exited with 1.

Prompted by a fish shell snippet that tried and failed to distinguish between
FreeBSD man (-p takes argument) and NetBSD man (-p no argument) by comparing
`man -p` exit code.

ok riastradh, logix (which also pointed out the manual page SYNOPSIS is
saying man -p should be used this way).

branches: 1.67.2; 1.67.4;
move 'utsname' to the main() function scope, so that the reference to
it outside the block remains valid.

should fix an asan reported issue.

branches: 1.66.2; 1.66.8;
Teach man -p to respect the MANPATH environment variable and the -M option.

Currently, `man -p` generates its output based on the value of the _default tag
in man.conf. However, man(1) modifies its search path based on the value of the
MANPATH variable and the list of directories specified via the -M option. In such
a case, `man -p` does not represent the correct search path. This commit intends
to fix this.

This change has the side effect that now the output of `man -p` will also include
the machine class specific subdirectories (such as man8/x86), while previously it
did not. The output would include subdirectories only for those machine classes
which are specified in the man.conf file.

Also, with this change, it is possible to run makemandb(8), by setting MANPATH
environment variable (e.g. env MANPATH=/usr/share/man makemandb).

Patch reviewed by wiz@

Fix comment indentation at couple of places: Use \t instead of space

branches: 1.64.6;
Document -f option for man(1).
Also remove unsupported options for `man -k` from the synopsis and usage.

Remove unused include and unused constant. Ok from christos@.

For an argument to be interpreted as a local file name, bypassing the
search rules in man.conf or MANPATH, it must begin with "/", "./", or
"../". Simply testing whether it contains "/" is wrong, because it
breaks usage like "man 8 vax/boot".

This reverts revision 1.57 dated 2013-10-06,
"Be more permissive in interpreting man pages as filenames".

branches: 1.61.4;
config() in manconf.c now verifies _build (and _crunch) command
templates with fmtcheck(3) so annotate the printf that uses these
commands as safe with a __format_arg wrapper and drop
-Wno-format-nonliteral.

XXX: Using local wrapper for now, solving this in general would be
nice, but it raises namespace pollution issues.

XXX^2: catman(8) also uses manconf.c and uses _build and _crunch so it
can also benefit from this (but see above).

Instead of guessing the suffix in the code, use the suffix list previously
loaded via man.conf(5). While there, zap unused iteration code.
(Franco Fichtner)

add more compression suffixes and local suffixes.

Recognize .gz and .bz2 suffixes so $ man ./man.1.gz works. From Franco Fichter
via dfly.

Be more permissive in interpreting man pages as filenames, from Franco
Fichter via dfly. fixes:
$ man usr.bin/man/man.1

Mark the dead.

Constify some more.

Spell "keywords" without space. Fix couple of typos.

manual_find_buildkeyword() - now that we control the format string, we
may use asterisk precision specification instead of temporary
modifying the _build string itself.

Fix manual_find_buildkeyword() to not use non-literal printf format.

Don't access memory outside the array if tmpdirlen == 0.

use -Wno-format and revert "fixstring"

Set the string to NUL instread of providing an new empty string (from uwe)

WARNS=6
- fix cast qual issues
- don't use snprintf on a user-provided buffer

Revert previous; other errors from WARNS=3 obscured the error in jump().

Constify jump() arguments appropriately.

Move manstate::pagerlen so that its "length of the above" comment makes sense.

branches: 1.44.6; 1.44.10;
If the default path doesn't result in a match, bail out early instead of
running into a segmentation fault. Based on patch by Abhinav Upadhyay.

branches: 1.43.2;
Fix usage in man page and sync usage in executable with it.

Add support for man -p to print the search path for manual pages (not
including cat page directories).

From Abhinav Upadhyay as part of Google's Summer of Code 2011

branches: 1.41.4;
get the value of the entry for the class not the key.

- centralize the snprintf code.
- use err where appropriate.
- add machclass which should be x86 when i386 and amd64 and can be specified
in man.conf as:
_i386 x86
_amd64 x86
so that we can support merged pages. Nothing uses this yet.

Mimic OSX behaviour:

On OS X it is possible to specify the manpage filename
with a full or relative path like this:

man ./foo.5

or

man /cd/foo/bar.1.gz

This is really helpful to view the manpage quickly while editing it.

patch presented on current-users@ and tech-userlevel@:
http://mail-index.netbsd.org/current-users/2009/10/06/msg010767.html
http://mail-index.netbsd.org/tech-userlevel/2009/10/06/msg002675.html

No objections

- use EXIT_FAILURE/EXIT_SUCCESS
- whitespace nits
- ansify cleanup()/usage()
- remove pointless parenthesis on return

branches: 1.37.4; 1.37.10;
Remove the \n and tabs from the __COPYRIGHT() strings.
Tweak to use a consistent format.

branches: 1.36.8;
Convert to using raise_default_signal(3).

branches: 1.35.4;
Since MAN_DEBUG is constant, go from if (MAN_DEBUG) to #if MAN_DEBUG

Coverity CID 2996: Don't return without closing fp.

modified man(1):
- avoid double slashes when displaying man pages (got tired
of '/usr/share/man//cat1/man.0').
- got rid of __P() while working on it.
- incorporate some of my old notes explaining how manpath works into the
comments of the code itself.
- renamed some of the vars so that the code is consistent throughout
(and hopefully clearer and easier to understand)
- fixed relative man paths for multiple man pages (man did a chdir()
on the first man page it had to format --- this broke any remaining
relative path man pages left to process). save old directory and
fchdir() back to it after formatting.
- improved doc on "man -h" which does more than just whatis(1) [e.g.
"man -h fopen" prints the required include files and the prototypes
rather than just the one-liner you get with whatis(1)]
- manconf.c now fills in the "len" length field in the TAG/ENTRY
structures (man now uses len).

revise man.conf file reading stuff to return error on failure in
addentry/gettag (fka getlist) rather than just err()ing out. this
allows man(1) to call cleanup and delete its tmp files rather than
just leave them floating. revise other apps using this code
(makewhatis, apropos, catman, whatis) to expect this. also remove
__P on updated files.

change an int to size_t

Homogenize usage messages: make the 'usage' word all lowercase, as this seems
to be the most common practice in our tree.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22365, verified by myself.

Rename man/config.{c,h} to man/manconf.{c,h}, so we don't have to
worry about problems including "config.h" when building as a host
tool.

- getlist(): add "int create" arg, which creates list if it's not present
(using guts of now defunct addlist())
- use TAILQ_*() macros appropriately when manipulating tailqs

branches: 1.27.2;
Fix longstanding misuse of glob(3) and fnmatch(3) in man page search.
glob(3)-special characters are now escaped in the supplied man page names.

Makes ``man ['' work without mysterious quoting.

convert to use getprogname()

Allow "-s" to specify a section name, ala the SysV man command.
Unlike the SysV man command, this doesn't allow for the -s argument
to be a list.

check_pager() was using an uninitialized variable; fix it.

allow to specify relative paths for sections in man.conf - they are used
similarily to _subdir, but only when appropriate
fix -m handling, so that e.g. "man -m . 3 printf" works as it should
add new -S flag, to specify a string the result path has to contain
g/c some unused stuff

Written by Chuck Cranor, with only cosmetic changes & const poisoning by me.

branches: 1.22.2;
Use uname(3) to determine MACHINE type.

revert previous

remove superfluous '-' in getopt string

branches: 1.19.2;
Try $TMPDIR first, then fall back to _PATH_TMP.

A caught signal is blocked in the handler and
must be unblocked to do the default action.
The old code may generate weird exit status
on signals.

Remove temp file on SIGPIPE (such as "man sh | head").

Fall back to more(1) if PAGER is set but null.

branches: 1.15.2;
char -> unsigned char

We can't define MACHINE from a makefile in Rhapsody, because you might be cross
compiling, or building a fat (multi-architecture) binary. Our compiler
defines __ARCHITECTURE__ for this use.
Fixed an off-by-one error that may occasionally cause man to seg fault. Found
by using a special strict malloc() implementation.

Add { and } to shut up egcs. Reformat the more questionable code.

Since GLOB_NOCHECK is not set, explicitly check for a return value of
GLOB_NOMATCH from glob(). Noticed by Juergen Hannken-Illjes.

GLOB_QUOTE is dead; per POSIX, backslash quoting of special characters being
enabled is the default behaviour.

When man was "fixed" to chdir() to the man root as historical man does,
it broke the -M option on relative paths (the chdir works as it should,
but the relative man page path is still used whole in nroff). This
repairs the situation by chopping the full pathname in half--one part
for the chdir, one part for the filename.

branches: 1.9.2;
cleanup for WARNS=1

resize a pathname buffer to MAXPATHLEN; from Matthias Scheler in PR 4101

Sync with 4.4BSD-Lite2

Pay attention to the section number, even if MANPATH or -M is specified.

Sync with 4.4-Lite, plus local changes.

Fix from Christos Zoulas which makes relative paths in .so lines work when
man pages are formatted.

fix from bostic for multiple match problem

fix from Keith White <kwhite@csi.UOttawa.CA>, to DTRT when man -w's output
is redirected.

branches: 1.1.1;
add man(1) ; part of new man(1) package from Bostic

man: remove unused global variable 'instype' (since yesterday)

No functional change.

man: remove trailing whitespace

No binary change.

man: fix type mismatch between enum and int (since yesterday)

No binary change.

man.c: fix -m option so it works as documented

Refactoring work in man.c r. 1.40 from twelve years ago introduced a
regression where input from the -m option was appended rather than
prepended to the search paths. Problem reported by C. Chapman on
netbsd-users.

Exit successfully after printing the search path, stop further processing.

Continuing to process had the unintended effect that `man` failed to find
a matching manual page for {EMPTY LIST OF REQUESTED MANUAL PAGES}, and
exited with 1.

Prompted by a fish shell snippet that tried and failed to distinguish between
FreeBSD man (-p takes argument) and NetBSD man (-p no argument) by comparing
`man -p` exit code.

ok riastradh, logix (which also pointed out the manual page SYNOPSIS is
saying man -p should be used this way).

branches: 1.67.2; 1.67.4;
move 'utsname' to the main() function scope, so that the reference to
it outside the block remains valid.

should fix an asan reported issue.

branches: 1.66.8;
Teach man -p to respect the MANPATH environment variable and the -M option.

Currently, `man -p` generates its output based on the value of the _default tag
in man.conf. However, man(1) modifies its search path based on the value of the
MANPATH variable and the list of directories specified via the -M option. In such
a case, `man -p` does not represent the correct search path. This commit intends
to fix this.

This change has the side effect that now the output of `man -p` will also include
the machine class specific subdirectories (such as man8/x86), while previously it
did not. The output would include subdirectories only for those machine classes
which are specified in the man.conf file.

Also, with this change, it is possible to run makemandb(8), by setting MANPATH
environment variable (e.g. env MANPATH=/usr/share/man makemandb).

Patch reviewed by wiz@

Fix comment indentation at couple of places: Use \t instead of space

branches: 1.64.6;
Document -f option for man(1).
Also remove unsupported options for `man -k` from the synopsis and usage.

Remove unused include and unused constant. Ok from christos@.

For an argument to be interpreted as a local file name, bypassing the
search rules in man.conf or MANPATH, it must begin with "/", "./", or
"../". Simply testing whether it contains "/" is wrong, because it
breaks usage like "man 8 vax/boot".

This reverts revision 1.57 dated 2013-10-06,
"Be more permissive in interpreting man pages as filenames".

branches: 1.61.4;
config() in manconf.c now verifies _build (and _crunch) command
templates with fmtcheck(3) so annotate the printf that uses these
commands as safe with a __format_arg wrapper and drop
-Wno-format-nonliteral.

XXX: Using local wrapper for now, solving this in general would be
nice, but it raises namespace pollution issues.

XXX^2: catman(8) also uses manconf.c and uses _build and _crunch so it
can also benefit from this (but see above).

Instead of guessing the suffix in the code, use the suffix list previously
loaded via man.conf(5). While there, zap unused iteration code.
(Franco Fichtner)

add more compression suffixes and local suffixes.

Recognize .gz and .bz2 suffixes so $ man ./man.1.gz works. From Franco Fichter
via dfly.

Be more permissive in interpreting man pages as filenames, from Franco
Fichter via dfly. fixes:
$ man usr.bin/man/man.1

Mark the dead.

Constify some more.

Spell "keywords" without space. Fix couple of typos.

manual_find_buildkeyword() - now that we control the format string, we
may use asterisk precision specification instead of temporary
modifying the _build string itself.

Fix manual_find_buildkeyword() to not use non-literal printf format.

Don't access memory outside the array if tmpdirlen == 0.

use -Wno-format and revert "fixstring"

Set the string to NUL instread of providing an new empty string (from uwe)

WARNS=6
- fix cast qual issues
- don't use snprintf on a user-provided buffer

Revert previous; other errors from WARNS=3 obscured the error in jump().

Constify jump() arguments appropriately.

Move manstate::pagerlen so that its "length of the above" comment makes sense.

branches: 1.44.6; 1.44.10;
If the default path doesn't result in a match, bail out early instead of
running into a segmentation fault. Based on patch by Abhinav Upadhyay.

branches: 1.43.2;
Fix usage in man page and sync usage in executable with it.

Add support for man -p to print the search path for manual pages (not
including cat page directories).

From Abhinav Upadhyay as part of Google's Summer of Code 2011

branches: 1.41.4;
get the value of the entry for the class not the key.

- centralize the snprintf code.
- use err where appropriate.
- add machclass which should be x86 when i386 and amd64 and can be specified
in man.conf as:
_i386 x86
_amd64 x86
so that we can support merged pages. Nothing uses this yet.

Mimic OSX behaviour:

On OS X it is possible to specify the manpage filename
with a full or relative path like this:

man ./foo.5

or

man /cd/foo/bar.1.gz

This is really helpful to view the manpage quickly while editing it.

patch presented on current-users@ and tech-userlevel@:
http://mail-index.netbsd.org/current-users/2009/10/06/msg010767.html
http://mail-index.netbsd.org/tech-userlevel/2009/10/06/msg002675.html

No objections

- use EXIT_FAILURE/EXIT_SUCCESS
- whitespace nits
- ansify cleanup()/usage()
- remove pointless parenthesis on return

branches: 1.37.4; 1.37.10;
Remove the \n and tabs from the __COPYRIGHT() strings.
Tweak to use a consistent format.

branches: 1.36.8;
Convert to using raise_default_signal(3).

branches: 1.35.4;
Since MAN_DEBUG is constant, go from if (MAN_DEBUG) to #if MAN_DEBUG

Coverity CID 2996: Don't return without closing fp.

modified man(1):
- avoid double slashes when displaying man pages (got tired
of '/usr/share/man//cat1/man.0').
- got rid of __P() while working on it.
- incorporate some of my old notes explaining how manpath works into the
comments of the code itself.
- renamed some of the vars so that the code is consistent throughout
(and hopefully clearer and easier to understand)
- fixed relative man paths for multiple man pages (man did a chdir()
on the first man page it had to format --- this broke any remaining
relative path man pages left to process). save old directory and
fchdir() back to it after formatting.
- improved doc on "man -h" which does more than just whatis(1) [e.g.
"man -h fopen" prints the required include files and the prototypes
rather than just the one-liner you get with whatis(1)]
- manconf.c now fills in the "len" length field in the TAG/ENTRY
structures (man now uses len).

revise man.conf file reading stuff to return error on failure in
addentry/gettag (fka getlist) rather than just err()ing out. this
allows man(1) to call cleanup and delete its tmp files rather than
just leave them floating. revise other apps using this code
(makewhatis, apropos, catman, whatis) to expect this. also remove
__P on updated files.

change an int to size_t

Homogenize usage messages: make the 'usage' word all lowercase, as this seems
to be the most common practice in our tree.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22365, verified by myself.

Rename man/config.{c,h} to man/manconf.{c,h}, so we don't have to
worry about problems including "config.h" when building as a host
tool.

- getlist(): add "int create" arg, which creates list if it's not present
(using guts of now defunct addlist())
- use TAILQ_*() macros appropriately when manipulating tailqs

branches: 1.27.2;
Fix longstanding misuse of glob(3) and fnmatch(3) in man page search.
glob(3)-special characters are now escaped in the supplied man page names.

Makes ``man ['' work without mysterious quoting.

convert to use getprogname()

Allow "-s" to specify a section name, ala the SysV man command.
Unlike the SysV man command, this doesn't allow for the -s argument
to be a list.

check_pager() was using an uninitialized variable; fix it.

allow to specify relative paths for sections in man.conf - they are used
similarily to _subdir, but only when appropriate
fix -m handling, so that e.g. "man -m . 3 printf" works as it should
add new -S flag, to specify a string the result path has to contain
g/c some unused stuff

Written by Chuck Cranor, with only cosmetic changes & const poisoning by me.

branches: 1.22.2;
Use uname(3) to determine MACHINE type.

revert previous

remove superfluous '-' in getopt string

branches: 1.19.2;
Try $TMPDIR first, then fall back to _PATH_TMP.

A caught signal is blocked in the handler and
must be unblocked to do the default action.
The old code may generate weird exit status
on signals.

Remove temp file on SIGPIPE (such as "man sh | head").

Fall back to more(1) if PAGER is set but null.

branches: 1.15.2;
char -> unsigned char

We can't define MACHINE from a makefile in Rhapsody, because you might be cross
compiling, or building a fat (multi-architecture) binary. Our compiler
defines __ARCHITECTURE__ for this use.
Fixed an off-by-one error that may occasionally cause man to seg fault. Found
by using a special strict malloc() implementation.

Add { and } to shut up egcs. Reformat the more questionable code.

Since GLOB_NOCHECK is not set, explicitly check for a return value of
GLOB_NOMATCH from glob(). Noticed by Juergen Hannken-Illjes.

GLOB_QUOTE is dead; per POSIX, backslash quoting of special characters being
enabled is the default behaviour.

When man was "fixed" to chdir() to the man root as historical man does,
it broke the -M option on relative paths (the chdir works as it should,
but the relative man page path is still used whole in nroff). This
repairs the situation by chopping the full pathname in half--one part
for the chdir, one part for the filename.

branches: 1.9.2;
cleanup for WARNS=1

resize a pathname buffer to MAXPATHLEN; from Matthias Scheler in PR 4101

Sync with 4.4BSD-Lite2

Pay attention to the section number, even if MANPATH or -M is specified.

Sync with 4.4-Lite, plus local changes.

Fix from Christos Zoulas which makes relative paths in .so lines work when
man pages are formatted.

fix from bostic for multiple match problem

fix from Keith White <kwhite@csi.UOttawa.CA>, to DTRT when man -w's output
is redirected.

branches: 1.1.1;
add man(1) ; part of new man(1) package from Bostic

man.c: fix -m option so it works as documented

Refactoring work in man.c r. 1.40 from twelve years ago introduced a
regression where input from the -m option was appended rather than
prepended to the search paths. Problem reported by C. Chapman on
netbsd-users.

Exit successfully after printing the search path, stop further processing.

Continuing to process had the unintended effect that `man` failed to find
a matching manual page for {EMPTY LIST OF REQUESTED MANUAL PAGES}, and
exited with 1.

Prompted by a fish shell snippet that tried and failed to distinguish between
FreeBSD man (-p takes argument) and NetBSD man (-p no argument) by comparing
`man -p` exit code.

ok riastradh, logix (which also pointed out the manual page SYNOPSIS is
saying man -p should be used this way).

branches: 1.67.2; 1.67.4;
move 'utsname' to the main() function scope, so that the reference to
it outside the block remains valid.

should fix an asan reported issue.

branches: 1.66.8;
Teach man -p to respect the MANPATH environment variable and the -M option.

Currently, `man -p` generates its output based on the value of the _default tag
in man.conf. However, man(1) modifies its search path based on the value of the
MANPATH variable and the list of directories specified via the -M option. In such
a case, `man -p` does not represent the correct search path. This commit intends
to fix this.

This change has the side effect that now the output of `man -p` will also include
the machine class specific subdirectories (such as man8/x86), while previously it
did not. The output would include subdirectories only for those machine classes
which are specified in the man.conf file.

Also, with this change, it is possible to run makemandb(8), by setting MANPATH
environment variable (e.g. env MANPATH=/usr/share/man makemandb).

Patch reviewed by wiz@

Fix comment indentation at couple of places: Use \t instead of space

branches: 1.64.6;
Document -f option for man(1).
Also remove unsupported options for `man -k` from the synopsis and usage.

Remove unused include and unused constant. Ok from christos@.

For an argument to be interpreted as a local file name, bypassing the
search rules in man.conf or MANPATH, it must begin with "/", "./", or
"../". Simply testing whether it contains "/" is wrong, because it
breaks usage like "man 8 vax/boot".

This reverts revision 1.57 dated 2013-10-06,
"Be more permissive in interpreting man pages as filenames".

branches: 1.61.4;
config() in manconf.c now verifies _build (and _crunch) command
templates with fmtcheck(3) so annotate the printf that uses these
commands as safe with a __format_arg wrapper and drop
-Wno-format-nonliteral.

XXX: Using local wrapper for now, solving this in general would be
nice, but it raises namespace pollution issues.

XXX^2: catman(8) also uses manconf.c and uses _build and _crunch so it
can also benefit from this (but see above).

Instead of guessing the suffix in the code, use the suffix list previously
loaded via man.conf(5). While there, zap unused iteration code.
(Franco Fichtner)

add more compression suffixes and local suffixes.

Recognize .gz and .bz2 suffixes so $ man ./man.1.gz works. From Franco Fichter
via dfly.

Be more permissive in interpreting man pages as filenames, from Franco
Fichter via dfly. fixes:
$ man usr.bin/man/man.1

Mark the dead.

Constify some more.

Spell "keywords" without space. Fix couple of typos.

manual_find_buildkeyword() - now that we control the format string, we
may use asterisk precision specification instead of temporary
modifying the _build string itself.

Fix manual_find_buildkeyword() to not use non-literal printf format.

Don't access memory outside the array if tmpdirlen == 0.

use -Wno-format and revert "fixstring"

Set the string to NUL instread of providing an new empty string (from uwe)

WARNS=6
- fix cast qual issues
- don't use snprintf on a user-provided buffer

Revert previous; other errors from WARNS=3 obscured the error in jump().

Constify jump() arguments appropriately.

Move manstate::pagerlen so that its "length of the above" comment makes sense.

branches: 1.44.6; 1.44.10;
If the default path doesn't result in a match, bail out early instead of
running into a segmentation fault. Based on patch by Abhinav Upadhyay.

branches: 1.43.2;
Fix usage in man page and sync usage in executable with it.

Add support for man -p to print the search path for manual pages (not
including cat page directories).

From Abhinav Upadhyay as part of Google's Summer of Code 2011

branches: 1.41.4;
get the value of the entry for the class not the key.

- centralize the snprintf code.
- use err where appropriate.
- add machclass which should be x86 when i386 and amd64 and can be specified
in man.conf as:
_i386 x86
_amd64 x86
so that we can support merged pages. Nothing uses this yet.

Mimic OSX behaviour:

On OS X it is possible to specify the manpage filename
with a full or relative path like this:

man ./foo.5

or

man /cd/foo/bar.1.gz

This is really helpful to view the manpage quickly while editing it.

patch presented on current-users@ and tech-userlevel@:
http://mail-index.netbsd.org/current-users/2009/10/06/msg010767.html
http://mail-index.netbsd.org/tech-userlevel/2009/10/06/msg002675.html

No objections

- use EXIT_FAILURE/EXIT_SUCCESS
- whitespace nits
- ansify cleanup()/usage()
- remove pointless parenthesis on return

branches: 1.37.4; 1.37.10;
Remove the \n and tabs from the __COPYRIGHT() strings.
Tweak to use a consistent format.

branches: 1.36.8;
Convert to using raise_default_signal(3).

branches: 1.35.4;
Since MAN_DEBUG is constant, go from if (MAN_DEBUG) to #if MAN_DEBUG

Coverity CID 2996: Don't return without closing fp.

modified man(1):
- avoid double slashes when displaying man pages (got tired
of '/usr/share/man//cat1/man.0').
- got rid of __P() while working on it.
- incorporate some of my old notes explaining how manpath works into the
comments of the code itself.
- renamed some of the vars so that the code is consistent throughout
(and hopefully clearer and easier to understand)
- fixed relative man paths for multiple man pages (man did a chdir()
on the first man page it had to format --- this broke any remaining
relative path man pages left to process). save old directory and
fchdir() back to it after formatting.
- improved doc on "man -h" which does more than just whatis(1) [e.g.
"man -h fopen" prints the required include files and the prototypes
rather than just the one-liner you get with whatis(1)]
- manconf.c now fills in the "len" length field in the TAG/ENTRY
structures (man now uses len).

revise man.conf file reading stuff to return error on failure in
addentry/gettag (fka getlist) rather than just err()ing out. this
allows man(1) to call cleanup and delete its tmp files rather than
just leave them floating. revise other apps using this code
(makewhatis, apropos, catman, whatis) to expect this. also remove
__P on updated files.

change an int to size_t

Homogenize usage messages: make the 'usage' word all lowercase, as this seems
to be the most common practice in our tree.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22365, verified by myself.

Rename man/config.{c,h} to man/manconf.{c,h}, so we don't have to
worry about problems including "config.h" when building as a host
tool.

- getlist(): add "int create" arg, which creates list if it's not present
(using guts of now defunct addlist())
- use TAILQ_*() macros appropriately when manipulating tailqs

branches: 1.27.2;
Fix longstanding misuse of glob(3) and fnmatch(3) in man page search.
glob(3)-special characters are now escaped in the supplied man page names.

Makes ``man ['' work without mysterious quoting.

convert to use getprogname()

Allow "-s" to specify a section name, ala the SysV man command.
Unlike the SysV man command, this doesn't allow for the -s argument
to be a list.

check_pager() was using an uninitialized variable; fix it.

allow to specify relative paths for sections in man.conf - they are used
similarily to _subdir, but only when appropriate
fix -m handling, so that e.g. "man -m . 3 printf" works as it should
add new -S flag, to specify a string the result path has to contain
g/c some unused stuff

Written by Chuck Cranor, with only cosmetic changes & const poisoning by me.

branches: 1.22.2;
Use uname(3) to determine MACHINE type.

revert previous

remove superfluous '-' in getopt string

branches: 1.19.2;
Try $TMPDIR first, then fall back to _PATH_TMP.

A caught signal is blocked in the handler and
must be unblocked to do the default action.
The old code may generate weird exit status
on signals.

Remove temp file on SIGPIPE (such as "man sh | head").

Fall back to more(1) if PAGER is set but null.

branches: 1.15.2;
char -> unsigned char

We can't define MACHINE from a makefile in Rhapsody, because you might be cross
compiling, or building a fat (multi-architecture) binary. Our compiler
defines __ARCHITECTURE__ for this use.
Fixed an off-by-one error that may occasionally cause man to seg fault. Found
by using a special strict malloc() implementation.

Add { and } to shut up egcs. Reformat the more questionable code.

Since GLOB_NOCHECK is not set, explicitly check for a return value of
GLOB_NOMATCH from glob(). Noticed by Juergen Hannken-Illjes.

GLOB_QUOTE is dead; per POSIX, backslash quoting of special characters being
enabled is the default behaviour.

When man was "fixed" to chdir() to the man root as historical man does,
it broke the -M option on relative paths (the chdir works as it should,
but the relative man page path is still used whole in nroff). This
repairs the situation by chopping the full pathname in half--one part
for the chdir, one part for the filename.

branches: 1.9.2;
cleanup for WARNS=1

resize a pathname buffer to MAXPATHLEN; from Matthias Scheler in PR 4101

Sync with 4.4BSD-Lite2

Pay attention to the section number, even if MANPATH or -M is specified.

Sync with 4.4-Lite, plus local changes.

Fix from Christos Zoulas which makes relative paths in .so lines work when
man pages are formatted.

fix from bostic for multiple match problem

fix from Keith White <kwhite@csi.UOttawa.CA>, to DTRT when man -w's output
is redirected.

branches: 1.1.1;
add man(1) ; part of new man(1) package from Bostic

Exit successfully after printing the search path, stop further processing.

Continuing to process had the unintended effect that `man` failed to find
a matching manual page for {EMPTY LIST OF REQUESTED MANUAL PAGES}, and
exited with 1.

Prompted by a fish shell snippet that tried and failed to distinguish between
FreeBSD man (-p takes argument) and NetBSD man (-p no argument) by comparing
`man -p` exit code.

ok riastradh, logix (which also pointed out the manual page SYNOPSIS is
saying man -p should be used this way).

move 'utsname' to the main() function scope, so that the reference to
it outside the block remains valid.

should fix an asan reported issue.

branches: 1.66.8;
Teach man -p to respect the MANPATH environment variable and the -M option.

Currently, `man -p` generates its output based on the value of the _default tag
in man.conf. However, man(1) modifies its search path based on the value of the
MANPATH variable and the list of directories specified via the -M option. In such
a case, `man -p` does not represent the correct search path. This commit intends
to fix this.

This change has the side effect that now the output of `man -p` will also include
the machine class specific subdirectories (such as man8/x86), while previously it
did not. The output would include subdirectories only for those machine classes
which are specified in the man.conf file.

Also, with this change, it is possible to run makemandb(8), by setting MANPATH
environment variable (e.g. env MANPATH=/usr/share/man makemandb).

Patch reviewed by wiz@

Fix comment indentation at couple of places: Use \t instead of space

branches: 1.64.6;
Document -f option for man(1).
Also remove unsupported options for `man -k` from the synopsis and usage.

Remove unused include and unused constant. Ok from christos@.

For an argument to be interpreted as a local file name, bypassing the
search rules in man.conf or MANPATH, it must begin with "/", "./", or
"../". Simply testing whether it contains "/" is wrong, because it
breaks usage like "man 8 vax/boot".

This reverts revision 1.57 dated 2013-10-06,
"Be more permissive in interpreting man pages as filenames".

branches: 1.61.4;
config() in manconf.c now verifies _build (and _crunch) command
templates with fmtcheck(3) so annotate the printf that uses these
commands as safe with a __format_arg wrapper and drop
-Wno-format-nonliteral.

XXX: Using local wrapper for now, solving this in general would be
nice, but it raises namespace pollution issues.

XXX^2: catman(8) also uses manconf.c and uses _build and _crunch so it
can also benefit from this (but see above).

Instead of guessing the suffix in the code, use the suffix list previously
loaded via man.conf(5). While there, zap unused iteration code.
(Franco Fichtner)

add more compression suffixes and local suffixes.

Recognize .gz and .bz2 suffixes so $ man ./man.1.gz works. From Franco Fichter
via dfly.

Be more permissive in interpreting man pages as filenames, from Franco
Fichter via dfly. fixes:
$ man usr.bin/man/man.1

Mark the dead.

Constify some more.

Spell "keywords" without space. Fix couple of typos.

manual_find_buildkeyword() - now that we control the format string, we
may use asterisk precision specification instead of temporary
modifying the _build string itself.

Fix manual_find_buildkeyword() to not use non-literal printf format.

Don't access memory outside the array if tmpdirlen == 0.

use -Wno-format and revert "fixstring"

Set the string to NUL instread of providing an new empty string (from uwe)

WARNS=6
- fix cast qual issues
- don't use snprintf on a user-provided buffer

Revert previous; other errors from WARNS=3 obscured the error in jump().

Constify jump() arguments appropriately.

Move manstate::pagerlen so that its "length of the above" comment makes sense.

branches: 1.44.6; 1.44.10;
If the default path doesn't result in a match, bail out early instead of
running into a segmentation fault. Based on patch by Abhinav Upadhyay.

branches: 1.43.2;
Fix usage in man page and sync usage in executable with it.

Add support for man -p to print the search path for manual pages (not
including cat page directories).

From Abhinav Upadhyay as part of Google's Summer of Code 2011

branches: 1.41.4;
get the value of the entry for the class not the key.

- centralize the snprintf code.
- use err where appropriate.
- add machclass which should be x86 when i386 and amd64 and can be specified
in man.conf as:
_i386 x86
_amd64 x86
so that we can support merged pages. Nothing uses this yet.

Mimic OSX behaviour:

On OS X it is possible to specify the manpage filename
with a full or relative path like this:

man ./foo.5

or

man /cd/foo/bar.1.gz

This is really helpful to view the manpage quickly while editing it.

patch presented on current-users@ and tech-userlevel@:
http://mail-index.netbsd.org/current-users/2009/10/06/msg010767.html
http://mail-index.netbsd.org/tech-userlevel/2009/10/06/msg002675.html

No objections

- use EXIT_FAILURE/EXIT_SUCCESS
- whitespace nits
- ansify cleanup()/usage()
- remove pointless parenthesis on return

branches: 1.37.4; 1.37.10;
Remove the \n and tabs from the __COPYRIGHT() strings.
Tweak to use a consistent format.

branches: 1.36.8;
Convert to using raise_default_signal(3).

branches: 1.35.4;
Since MAN_DEBUG is constant, go from if (MAN_DEBUG) to #if MAN_DEBUG

Coverity CID 2996: Don't return without closing fp.

modified man(1):
- avoid double slashes when displaying man pages (got tired
of '/usr/share/man//cat1/man.0').
- got rid of __P() while working on it.
- incorporate some of my old notes explaining how manpath works into the
comments of the code itself.
- renamed some of the vars so that the code is consistent throughout
(and hopefully clearer and easier to understand)
- fixed relative man paths for multiple man pages (man did a chdir()
on the first man page it had to format --- this broke any remaining
relative path man pages left to process). save old directory and
fchdir() back to it after formatting.
- improved doc on "man -h" which does more than just whatis(1) [e.g.
"man -h fopen" prints the required include files and the prototypes
rather than just the one-liner you get with whatis(1)]
- manconf.c now fills in the "len" length field in the TAG/ENTRY
structures (man now uses len).

revise man.conf file reading stuff to return error on failure in
addentry/gettag (fka getlist) rather than just err()ing out. this
allows man(1) to call cleanup and delete its tmp files rather than
just leave them floating. revise other apps using this code
(makewhatis, apropos, catman, whatis) to expect this. also remove
__P on updated files.

change an int to size_t

Homogenize usage messages: make the 'usage' word all lowercase, as this seems
to be the most common practice in our tree.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22365, verified by myself.

Rename man/config.{c,h} to man/manconf.{c,h}, so we don't have to
worry about problems including "config.h" when building as a host
tool.

- getlist(): add "int create" arg, which creates list if it's not present
(using guts of now defunct addlist())
- use TAILQ_*() macros appropriately when manipulating tailqs

branches: 1.27.2;
Fix longstanding misuse of glob(3) and fnmatch(3) in man page search.
glob(3)-special characters are now escaped in the supplied man page names.

Makes ``man ['' work without mysterious quoting.

convert to use getprogname()

Allow "-s" to specify a section name, ala the SysV man command.
Unlike the SysV man command, this doesn't allow for the -s argument
to be a list.

check_pager() was using an uninitialized variable; fix it.

allow to specify relative paths for sections in man.conf - they are used
similarily to _subdir, but only when appropriate
fix -m handling, so that e.g. "man -m . 3 printf" works as it should
add new -S flag, to specify a string the result path has to contain
g/c some unused stuff

Written by Chuck Cranor, with only cosmetic changes & const poisoning by me.

branches: 1.22.2;
Use uname(3) to determine MACHINE type.

revert previous

remove superfluous '-' in getopt string

branches: 1.19.2;
Try $TMPDIR first, then fall back to _PATH_TMP.

A caught signal is blocked in the handler and
must be unblocked to do the default action.
The old code may generate weird exit status
on signals.

Remove temp file on SIGPIPE (such as "man sh | head").

Fall back to more(1) if PAGER is set but null.

branches: 1.15.2;
char -> unsigned char

We can't define MACHINE from a makefile in Rhapsody, because you might be cross
compiling, or building a fat (multi-architecture) binary. Our compiler
defines __ARCHITECTURE__ for this use.
Fixed an off-by-one error that may occasionally cause man to seg fault. Found
by using a special strict malloc() implementation.

Add { and } to shut up egcs. Reformat the more questionable code.

Since GLOB_NOCHECK is not set, explicitly check for a return value of
GLOB_NOMATCH from glob(). Noticed by Juergen Hannken-Illjes.

GLOB_QUOTE is dead; per POSIX, backslash quoting of special characters being
enabled is the default behaviour.

When man was "fixed" to chdir() to the man root as historical man does,
it broke the -M option on relative paths (the chdir works as it should,
but the relative man page path is still used whole in nroff). This
repairs the situation by chopping the full pathname in half--one part
for the chdir, one part for the filename.

branches: 1.9.2;
cleanup for WARNS=1

resize a pathname buffer to MAXPATHLEN; from Matthias Scheler in PR 4101

Sync with 4.4BSD-Lite2

Pay attention to the section number, even if MANPATH or -M is specified.

Sync with 4.4-Lite, plus local changes.

Fix from Christos Zoulas which makes relative paths in .so lines work when
man pages are formatted.

fix from bostic for multiple match problem

fix from Keith White <kwhite@csi.UOttawa.CA>, to DTRT when man -w's output
is redirected.

branches: 1.1.1;
add man(1) ; part of new man(1) package from Bostic

move 'utsname' to the main() function scope, so that the reference to
it outside the block remains valid.

should fix an asan reported issue.

branches: 1.66.8;
Teach man -p to respect the MANPATH environment variable and the -M option.

Currently, `man -p` generates its output based on the value of the _default tag
in man.conf. However, man(1) modifies its search path based on the value of the
MANPATH variable and the list of directories specified via the -M option. In such
a case, `man -p` does not represent the correct search path. This commit intends
to fix this.

This change has the side effect that now the output of `man -p` will also include
the machine class specific subdirectories (such as man8/x86), while previously it
did not. The output would include subdirectories only for those machine classes
which are specified in the man.conf file.

Also, with this change, it is possible to run makemandb(8), by setting MANPATH
environment variable (e.g. env MANPATH=/usr/share/man makemandb).

Patch reviewed by wiz@

Fix comment indentation at couple of places: Use \t instead of space

branches: 1.64.6;
Document -f option for man(1).
Also remove unsupported options for `man -k` from the synopsis and usage.

Remove unused include and unused constant. Ok from christos@.

For an argument to be interpreted as a local file name, bypassing the
search rules in man.conf or MANPATH, it must begin with "/", "./", or
"../". Simply testing whether it contains "/" is wrong, because it
breaks usage like "man 8 vax/boot".

This reverts revision 1.57 dated 2013-10-06,
"Be more permissive in interpreting man pages as filenames".

branches: 1.61.4;
config() in manconf.c now verifies _build (and _crunch) command
templates with fmtcheck(3) so annotate the printf that uses these
commands as safe with a __format_arg wrapper and drop
-Wno-format-nonliteral.

XXX: Using local wrapper for now, solving this in general would be
nice, but it raises namespace pollution issues.

XXX^2: catman(8) also uses manconf.c and uses _build and _crunch so it
can also benefit from this (but see above).

Instead of guessing the suffix in the code, use the suffix list previously
loaded via man.conf(5). While there, zap unused iteration code.
(Franco Fichtner)

add more compression suffixes and local suffixes.

Recognize .gz and .bz2 suffixes so $ man ./man.1.gz works. From Franco Fichter
via dfly.

Be more permissive in interpreting man pages as filenames, from Franco
Fichter via dfly. fixes:
$ man usr.bin/man/man.1

Mark the dead.

Constify some more.

Spell "keywords" without space. Fix couple of typos.

manual_find_buildkeyword() - now that we control the format string, we
may use asterisk precision specification instead of temporary
modifying the _build string itself.

Fix manual_find_buildkeyword() to not use non-literal printf format.

Don't access memory outside the array if tmpdirlen == 0.

use -Wno-format and revert "fixstring"

Set the string to NUL instread of providing an new empty string (from uwe)

WARNS=6
- fix cast qual issues
- don't use snprintf on a user-provided buffer

Revert previous; other errors from WARNS=3 obscured the error in jump().

Constify jump() arguments appropriately.

Move manstate::pagerlen so that its "length of the above" comment makes sense.

branches: 1.44.6; 1.44.10;
If the default path doesn't result in a match, bail out early instead of
running into a segmentation fault. Based on patch by Abhinav Upadhyay.

branches: 1.43.2;
Fix usage in man page and sync usage in executable with it.

Add support for man -p to print the search path for manual pages (not
including cat page directories).

From Abhinav Upadhyay as part of Google's Summer of Code 2011

branches: 1.41.4;
get the value of the entry for the class not the key.

- centralize the snprintf code.
- use err where appropriate.
- add machclass which should be x86 when i386 and amd64 and can be specified
in man.conf as:
_i386 x86
_amd64 x86
so that we can support merged pages. Nothing uses this yet.

Mimic OSX behaviour:

On OS X it is possible to specify the manpage filename
with a full or relative path like this:

man ./foo.5

or

man /cd/foo/bar.1.gz

This is really helpful to view the manpage quickly while editing it.

patch presented on current-users@ and tech-userlevel@:
http://mail-index.netbsd.org/current-users/2009/10/06/msg010767.html
http://mail-index.netbsd.org/tech-userlevel/2009/10/06/msg002675.html

No objections

- use EXIT_FAILURE/EXIT_SUCCESS
- whitespace nits
- ansify cleanup()/usage()
- remove pointless parenthesis on return

branches: 1.37.4; 1.37.10;
Remove the \n and tabs from the __COPYRIGHT() strings.
Tweak to use a consistent format.

branches: 1.36.8;
Convert to using raise_default_signal(3).

branches: 1.35.4;
Since MAN_DEBUG is constant, go from if (MAN_DEBUG) to #if MAN_DEBUG

Coverity CID 2996: Don't return without closing fp.

modified man(1):
- avoid double slashes when displaying man pages (got tired
of '/usr/share/man//cat1/man.0').
- got rid of __P() while working on it.
- incorporate some of my old notes explaining how manpath works into the
comments of the code itself.
- renamed some of the vars so that the code is consistent throughout
(and hopefully clearer and easier to understand)
- fixed relative man paths for multiple man pages (man did a chdir()
on the first man page it had to format --- this broke any remaining
relative path man pages left to process). save old directory and
fchdir() back to it after formatting.
- improved doc on "man -h" which does more than just whatis(1) [e.g.
"man -h fopen" prints the required include files and the prototypes
rather than just the one-liner you get with whatis(1)]
- manconf.c now fills in the "len" length field in the TAG/ENTRY
structures (man now uses len).

revise man.conf file reading stuff to return error on failure in
addentry/gettag (fka getlist) rather than just err()ing out. this
allows man(1) to call cleanup and delete its tmp files rather than
just leave them floating. revise other apps using this code
(makewhatis, apropos, catman, whatis) to expect this. also remove
__P on updated files.

change an int to size_t

Homogenize usage messages: make the 'usage' word all lowercase, as this seems
to be the most common practice in our tree.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22365, verified by myself.

Rename man/config.{c,h} to man/manconf.{c,h}, so we don't have to
worry about problems including "config.h" when building as a host
tool.

- getlist(): add "int create" arg, which creates list if it's not present
(using guts of now defunct addlist())
- use TAILQ_*() macros appropriately when manipulating tailqs

branches: 1.27.2;
Fix longstanding misuse of glob(3) and fnmatch(3) in man page search.
glob(3)-special characters are now escaped in the supplied man page names.

Makes ``man ['' work without mysterious quoting.

convert to use getprogname()

Allow "-s" to specify a section name, ala the SysV man command.
Unlike the SysV man command, this doesn't allow for the -s argument
to be a list.

check_pager() was using an uninitialized variable; fix it.

allow to specify relative paths for sections in man.conf - they are used
similarily to _subdir, but only when appropriate
fix -m handling, so that e.g. "man -m . 3 printf" works as it should
add new -S flag, to specify a string the result path has to contain
g/c some unused stuff

Written by Chuck Cranor, with only cosmetic changes & const poisoning by me.

branches: 1.22.2;
Use uname(3) to determine MACHINE type.

revert previous

remove superfluous '-' in getopt string

branches: 1.19.2;
Try $TMPDIR first, then fall back to _PATH_TMP.

A caught signal is blocked in the handler and
must be unblocked to do the default action.
The old code may generate weird exit status
on signals.

Remove temp file on SIGPIPE (such as "man sh | head").

Fall back to more(1) if PAGER is set but null.

branches: 1.15.2;
char -> unsigned char

We can't define MACHINE from a makefile in Rhapsody, because you might be cross
compiling, or building a fat (multi-architecture) binary. Our compiler
defines __ARCHITECTURE__ for this use.
Fixed an off-by-one error that may occasionally cause man to seg fault. Found
by using a special strict malloc() implementation.

Add { and } to shut up egcs. Reformat the more questionable code.

Since GLOB_NOCHECK is not set, explicitly check for a return value of
GLOB_NOMATCH from glob(). Noticed by Juergen Hannken-Illjes.

GLOB_QUOTE is dead; per POSIX, backslash quoting of special characters being
enabled is the default behaviour.

When man was "fixed" to chdir() to the man root as historical man does,
it broke the -M option on relative paths (the chdir works as it should,
but the relative man page path is still used whole in nroff). This
repairs the situation by chopping the full pathname in half--one part
for the chdir, one part for the filename.

branches: 1.9.2;
cleanup for WARNS=1

resize a pathname buffer to MAXPATHLEN; from Matthias Scheler in PR 4101

Sync with 4.4BSD-Lite2

Pay attention to the section number, even if MANPATH or -M is specified.

Sync with 4.4-Lite, plus local changes.

Fix from Christos Zoulas which makes relative paths in .so lines work when
man pages are formatted.

fix from bostic for multiple match problem

fix from Keith White <kwhite@csi.UOttawa.CA>, to DTRT when man -w's output
is redirected.

branches: 1.1.1;
add man(1) ; part of new man(1) package from Bostic

Teach man -p to respect the MANPATH environment variable and the -M option.

Currently, `man -p` generates its output based on the value of the _default tag
in man.conf. However, man(1) modifies its search path based on the value of the
MANPATH variable and the list of directories specified via the -M option. In such
a case, `man -p` does not represent the correct search path. This commit intends
to fix this.

This change has the side effect that now the output of `man -p` will also include
the machine class specific subdirectories (such as man8/x86), while previously it
did not. The output would include subdirectories only for those machine classes
which are specified in the man.conf file.

Also, with this change, it is possible to run makemandb(8), by setting MANPATH
environment variable (e.g. env MANPATH=/usr/share/man makemandb).

Patch reviewed by wiz@

Fix comment indentation at couple of places: Use \t instead of space

branches: 1.64.6;
Document -f option for man(1).
Also remove unsupported options for `man -k` from the synopsis and usage.

Remove unused include and unused constant. Ok from christos@.

For an argument to be interpreted as a local file name, bypassing the
search rules in man.conf or MANPATH, it must begin with "/", "./", or
"../". Simply testing whether it contains "/" is wrong, because it
breaks usage like "man 8 vax/boot".

This reverts revision 1.57 dated 2013-10-06,
"Be more permissive in interpreting man pages as filenames".

branches: 1.61.4;
config() in manconf.c now verifies _build (and _crunch) command
templates with fmtcheck(3) so annotate the printf that uses these
commands as safe with a __format_arg wrapper and drop
-Wno-format-nonliteral.

XXX: Using local wrapper for now, solving this in general would be
nice, but it raises namespace pollution issues.

XXX^2: catman(8) also uses manconf.c and uses _build and _crunch so it
can also benefit from this (but see above).

Instead of guessing the suffix in the code, use the suffix list previously
loaded via man.conf(5). While there, zap unused iteration code.
(Franco Fichtner)

add more compression suffixes and local suffixes.

Recognize .gz and .bz2 suffixes so $ man ./man.1.gz works. From Franco Fichter
via dfly.

Be more permissive in interpreting man pages as filenames, from Franco
Fichter via dfly. fixes:
$ man usr.bin/man/man.1

Mark the dead.

Constify some more.

Spell "keywords" without space. Fix couple of typos.

manual_find_buildkeyword() - now that we control the format string, we
may use asterisk precision specification instead of temporary
modifying the _build string itself.

Fix manual_find_buildkeyword() to not use non-literal printf format.

Don't access memory outside the array if tmpdirlen == 0.

use -Wno-format and revert "fixstring"

Set the string to NUL instread of providing an new empty string (from uwe)

WARNS=6
- fix cast qual issues
- don't use snprintf on a user-provided buffer

Revert previous; other errors from WARNS=3 obscured the error in jump().

Constify jump() arguments appropriately.

Move manstate::pagerlen so that its "length of the above" comment makes sense.

branches: 1.44.6; 1.44.10;
If the default path doesn't result in a match, bail out early instead of
running into a segmentation fault. Based on patch by Abhinav Upadhyay.

branches: 1.43.2;
Fix usage in man page and sync usage in executable with it.

Add support for man -p to print the search path for manual pages (not
including cat page directories).

From Abhinav Upadhyay as part of Google's Summer of Code 2011

branches: 1.41.4;
get the value of the entry for the class not the key.

- centralize the snprintf code.
- use err where appropriate.
- add machclass which should be x86 when i386 and amd64 and can be specified
in man.conf as:
_i386 x86
_amd64 x86
so that we can support merged pages. Nothing uses this yet.

Mimic OSX behaviour:

On OS X it is possible to specify the manpage filename
with a full or relative path like this:

man ./foo.5

or

man /cd/foo/bar.1.gz

This is really helpful to view the manpage quickly while editing it.

patch presented on current-users@ and tech-userlevel@:
http://mail-index.netbsd.org/current-users/2009/10/06/msg010767.html
http://mail-index.netbsd.org/tech-userlevel/2009/10/06/msg002675.html

No objections

- use EXIT_FAILURE/EXIT_SUCCESS
- whitespace nits
- ansify cleanup()/usage()
- remove pointless parenthesis on return

branches: 1.37.4; 1.37.10;
Remove the \n and tabs from the __COPYRIGHT() strings.
Tweak to use a consistent format.

branches: 1.36.8;
Convert to using raise_default_signal(3).

branches: 1.35.4;
Since MAN_DEBUG is constant, go from if (MAN_DEBUG) to #if MAN_DEBUG

Coverity CID 2996: Don't return without closing fp.

modified man(1):
- avoid double slashes when displaying man pages (got tired
of '/usr/share/man//cat1/man.0').
- got rid of __P() while working on it.
- incorporate some of my old notes explaining how manpath works into the
comments of the code itself.
- renamed some of the vars so that the code is consistent throughout
(and hopefully clearer and easier to understand)
- fixed relative man paths for multiple man pages (man did a chdir()
on the first man page it had to format --- this broke any remaining
relative path man pages left to process). save old directory and
fchdir() back to it after formatting.
- improved doc on "man -h" which does more than just whatis(1) [e.g.
"man -h fopen" prints the required include files and the prototypes
rather than just the one-liner you get with whatis(1)]
- manconf.c now fills in the "len" length field in the TAG/ENTRY
structures (man now uses len).

revise man.conf file reading stuff to return error on failure in
addentry/gettag (fka getlist) rather than just err()ing out. this
allows man(1) to call cleanup and delete its tmp files rather than
just leave them floating. revise other apps using this code
(makewhatis, apropos, catman, whatis) to expect this. also remove
__P on updated files.

change an int to size_t

Homogenize usage messages: make the 'usage' word all lowercase, as this seems
to be the most common practice in our tree.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22365, verified by myself.

Rename man/config.{c,h} to man/manconf.{c,h}, so we don't have to
worry about problems including "config.h" when building as a host
tool.

- getlist(): add "int create" arg, which creates list if it's not present
(using guts of now defunct addlist())
- use TAILQ_*() macros appropriately when manipulating tailqs

branches: 1.27.2;
Fix longstanding misuse of glob(3) and fnmatch(3) in man page search.
glob(3)-special characters are now escaped in the supplied man page names.

Makes ``man ['' work without mysterious quoting.

convert to use getprogname()

Allow "-s" to specify a section name, ala the SysV man command.
Unlike the SysV man command, this doesn't allow for the -s argument
to be a list.

check_pager() was using an uninitialized variable; fix it.

allow to specify relative paths for sections in man.conf - they are used
similarily to _subdir, but only when appropriate
fix -m handling, so that e.g. "man -m . 3 printf" works as it should
add new -S flag, to specify a string the result path has to contain
g/c some unused stuff

Written by Chuck Cranor, with only cosmetic changes & const poisoning by me.

branches: 1.22.2;
Use uname(3) to determine MACHINE type.

revert previous

remove superfluous '-' in getopt string

branches: 1.19.2;
Try $TMPDIR first, then fall back to _PATH_TMP.

A caught signal is blocked in the handler and
must be unblocked to do the default action.
The old code may generate weird exit status
on signals.

Remove temp file on SIGPIPE (such as "man sh | head").

Fall back to more(1) if PAGER is set but null.

branches: 1.15.2;
char -> unsigned char

We can't define MACHINE from a makefile in Rhapsody, because you might be cross
compiling, or building a fat (multi-architecture) binary. Our compiler
defines __ARCHITECTURE__ for this use.
Fixed an off-by-one error that may occasionally cause man to seg fault. Found
by using a special strict malloc() implementation.

Add { and } to shut up egcs. Reformat the more questionable code.

Since GLOB_NOCHECK is not set, explicitly check for a return value of
GLOB_NOMATCH from glob(). Noticed by Juergen Hannken-Illjes.

GLOB_QUOTE is dead; per POSIX, backslash quoting of special characters being
enabled is the default behaviour.

When man was "fixed" to chdir() to the man root as historical man does,
it broke the -M option on relative paths (the chdir works as it should,
but the relative man page path is still used whole in nroff). This
repairs the situation by chopping the full pathname in half--one part
for the chdir, one part for the filename.

branches: 1.9.2;
cleanup for WARNS=1

resize a pathname buffer to MAXPATHLEN; from Matthias Scheler in PR 4101

Sync with 4.4BSD-Lite2

Pay attention to the section number, even if MANPATH or -M is specified.

Sync with 4.4-Lite, plus local changes.

Fix from Christos Zoulas which makes relative paths in .so lines work when
man pages are formatted.

fix from bostic for multiple match problem

fix from Keith White <kwhite@csi.UOttawa.CA>, to DTRT when man -w's output
is redirected.

branches: 1.1.1;
add man(1) ; part of new man(1) package from Bostic

Fix comment indentation at couple of places: Use \t instead of space

Document -f option for man(1).
Also remove unsupported options for `man -k` from the synopsis and usage.

Remove unused include and unused constant. Ok from christos@.

For an argument to be interpreted as a local file name, bypassing the
search rules in man.conf or MANPATH, it must begin with "/", "./", or
"../". Simply testing whether it contains "/" is wrong, because it
breaks usage like "man 8 vax/boot".

This reverts revision 1.57 dated 2013-10-06,
"Be more permissive in interpreting man pages as filenames".

branches: 1.61.4;
config() in manconf.c now verifies _build (and _crunch) command
templates with fmtcheck(3) so annotate the printf that uses these
commands as safe with a __format_arg wrapper and drop
-Wno-format-nonliteral.

XXX: Using local wrapper for now, solving this in general would be
nice, but it raises namespace pollution issues.

XXX^2: catman(8) also uses manconf.c and uses _build and _crunch so it
can also benefit from this (but see above).

Instead of guessing the suffix in the code, use the suffix list previously
loaded via man.conf(5). While there, zap unused iteration code.
(Franco Fichtner)

add more compression suffixes and local suffixes.

Recognize .gz and .bz2 suffixes so $ man ./man.1.gz works. From Franco Fichter
via dfly.

Be more permissive in interpreting man pages as filenames, from Franco
Fichter via dfly. fixes:
$ man usr.bin/man/man.1