Re: apropos not working after /etc/weekly is run

[Iain Hibbert <plunky%rya-online.net@localhost>]

On Tue, 14 Feb 2012, Joerg Sonnenberger wrote:

> On Tue, Feb 14, 2012 at 10:00:23PM +0000, Iain Hibbert wrote:
> > Any comments before I send this to releng?
>
> A patch for this is pending.

well, I see

> Your patch doesn't fix the issue, in fact in actively makes the
> situation worse.

without some justification this comment is useless. The facts are

1. "makemandb -f" removes the file and recreates it from scratch
2. having "umask 077" in effect forces the file to be unreadable

my patch changes the behaviour so that the file is "pruned" as per the
documentation, rather than "removed and recreated" and it does fix that..

for example, using makemandb with my patch

# umask 022
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47801344 Feb 15 03:46 /var/db/man.db
# makemandb
[...]
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47801344 Feb 15 03:46 /var/db/man.db
# makemandb -f
[...]
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47799296 Feb 15 08:01 /var/db/man.db

# umask 077
# makemandb
[...]
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47799296 Feb 15 08:01 /var/db/man.db
# makemandb -f
[...]
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47799296 Feb 15 08:03 /var/db/man.db

works as expected, but using -current makemandb

# umask 022
# makemandb
[...]
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47799296 Feb 15 08:03 /var/db/man.db
# makemandb -f
[...]
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47799296 Feb 15 08:06 /var/db/man.db

# umask 077
# makemandb
[...]
# ls -l /var/db/man.db
-rw-r--r--  1 root  wheel  47799296 Feb 15 08:06 /var/db/man.db
# makemandb -f
[...]
# ls -l /var/db/man.db
-rw-------  1 root  wheel  47799296 Feb 15 08:08 /var/db/man.db

makes the file unreadable when umask 077 is in effect.

So now, please explain - how is it that

1. this does not fix the issue
2. this makes it actively worse

?

regards,
iain