The / after $p is sensible (although /usr/man and /usr/share/man are both in
the path I saw, which is harmless but redundant). The formatting...ok,
everybody has their conventions. Granted that wc and tr are less efficient that
what you suggest, in rare cases (like if lots of man pages were
On Tue, 21 Sep 2021, Richard L. Hamilton wrote:
; Most of the time, man indexes don't actually need to be rebuilt. The attached
change to /lib/svc/method/update-man-index should (I think) only update the
directories (if any) that are not current, so that most of the time, it
finishes in a
Perhaps better, to be consistent with what I see elsewhere in method scripts,
one line should be further changed to use full paths of executables:
if [ -d "${p}" -a -f "${p}/whatis" ] && [ "$(/usr/bin/find "${p}"
-newer "${p}/whatis" | /usr/bin/wc -l | /usr/bin/tr -d ' ')" == "0" ]; then
Most of the time, man indexes don't actually need to be rebuilt. The attached
change to /lib/svc/method/update-man-index should (I think) only update the
directories (if any) that are not current, so that most of the time, it
finishes in a second or so rather than a couple of minutes (less