NetBSD/usr.sbin/catman
lukem c4b7a9e794 bsd.own.mk: rename GCC_NO_* to CC_WNO_*
Rename compiler-warning-disable variables from
	GCC_NO_warning
to
	CC_WNO_warning
where warning is the full warning name as used by the compiler.

GCC_NO_IMPLICIT_FALLTHRU is CC_WNO_IMPLICIT_FALLTHROUGH

Using the convention CC_compilerflag, where compilerflag
is based on the full compiler flag name.
2023-06-03 09:09:01 +00:00
..
Makefile bsd.own.mk: rename GCC_NO_* to CC_WNO_* 2023-06-03 09:09:01 +00:00
README.hardlinks
TODO
catman.8 Restore BSD BUGS section 2021-08-15 11:00:43 +00:00
catman.c enlarge buffer sizes 2019-10-12 17:26:26 +00:00
pathnames.h

README.hardlinks

# $NetBSD: README.hardlinks,v 1.1 2016/05/29 22:32:03 dholland Exp $
#

catman.8 notes that this code doesn't handle hard links.

To fix this, one might proceed as follows:

(1) Add an additional data structure mapping (fsid_t, ino_t) pairs
from stat to filenames.

(2) In scanmandir(), in the readdir loop, next to the code that checks
for a symlink, check if the page's linkcount > 1 (manstat.st_nlink > 1)
and if so:
   a. Check the new data structure to see if this file's been seen before.
      If so, use the saved name of the page to construct a hard link in the
      cat directory, and continue to the next directory entry.
   b. If not, add to the new data structure to remember this page and
      process it as normal.

One might clear the data structure for each mandir or not; probably
for each man tree is best. The size of the structure isn't
prohibitive; but the chances of ever seeing the same hardlinked page
in two different man trees (e.g. in both /usr/share/man and
/usr/pkg/man) are pretty well zero.

All of this seems like a SMOP, but it doesn't really seem worth doing
at the moment given that we don't build catpages at all by default and
they aren't particularly useful to have any more except on the slowest
of slow hardware. I've left this note so that someone else can take it
up if they see fit.

 - dholland 20160529