Adapt compressdoc script to Man-DB

[alexander@…]

The compressdoc script from ​http://www.linuxfromscratch.org/blfs/view/svn/postlfs/compressdoc.html reads the manual page paths from /etc/man.conf. This file no longer exists, because the LFS book uses Man-DB.

[dnicholson@…]

Alexander, the patch is gone somehow. Could you resubmit it?

[alexander@…]

Attachment re-added

[dnicholson@…]

OK, I've got the patch now. This is how I'm going to handle it. Since LFS stable still uses man, I'll add the patch to the repo, and add a note on the compressdoc page to use this patch if you have Man-DB. When LFS stable uses Man-DB, I'll make it the default.

Question, where is the appropriate patches location to add this patch since it's not really for a package?

[archaic@…]

This patch can break in certain environments because it reads man_db.conf unconditionally and tries to change into dirs that don't exist. Use of the man-db utility "manpath" is a very easy drop-in replacement for man --path -C. I recommend the patch be regenerated so that the following is removed:

-  MAN_DIR=`man --path -C "$MAN_CONF"/man.conf \
-            | sed 's/:/\\n/g' \
+  MAN_DIR=`sed -n 's@^MANDATORY_MANPATH[^/]*\(/.*\)$@\1@ p' \
+              "$MAN_CONF"/man_db.conf \

and becomes:

-  MAN_DIR=`man --path -C "$MAN_CONF"/man.conf \
+  MAN_DIR=`manpath -C "$MAN_CONF"/man_db.conf \

The line with the sed from the original patch can stay unchanged because manpath outputs the same format as man --path does.

[dnicholson@…]

Alexander, I'm assuming that since you haven't said anything that you're OK with the changes Archaic has suggested. I've generated a new patch that fixes a couple other output nitpicks I found when I actually tried to use it. I think the `manpath' suggestion works well.

[dnicholson@…]

Patch using `manpath' utility

[alexander@…]

The script using the "manpath" utility works. Thanks.

[dnicholson@…]

Committed in r5735. Closing ticket.