Re: bin/46419: Inconsistent information in man.db for man page aliases after updation

[Abhinav Upadhyay <er.abhinav.upadhyay%gmail.com@localhost>]

On Mon, May 7, 2012 at 6:15 AM, Matthew Mondor 
<mm_lists%pulsar-zone.net@localhost> wrote:
> The following reply was made to PR bin/46419; it has been noted by GNATS.
>
> From: Matthew Mondor <mm_lists%pulsar-zone.net@localhost>
> To: gnats-bugs%NetBSD.org@localhost
> Cc:
> Subject: Re: bin/46419: Inconsistent information in man.db for man page
>  aliases after updation
> Date: Sun, 6 May 2012 20:44:57 -0400
>
>  On Sun,  6 May 2012 19:00:01 +0000 (UTC)
>  er.abhinav.upadhyay%gmail.com@localhost wrote:
>
>  > makemandb(8) maintains an index of all the different aliases of the man 
> pages. This information is maintained in the mandb_links table in the 
> database. But there is a small bug in there. When old man pages are removed, 
> the corresponding entry for the aliases of those man pages are not removed 
> from the mandb_links table.
>
>  Sorry that I didn't yet read the whole implementation, thus this
>  question:
>
>  I see that a new column is added to the new table, which gets created
>  if it doesn't exist.  Is this table deleted and recreated at every
>  makemandb run?  Otherwise, some migration code might be needed to alter
>  the existing table and add the new column (or to delete the existing
>  tables if they're not of the intended version/format)...
>
>  Or perhaps this doesn't matter as there was no official release yet
>  with the new code, and UPDATING could hold a note with instructions for
>  the admins to delete the old db (if that is a problem, that is)?
>
>  Thanks,
>  --

Yes, good catch. I should also change the schema version in
apropos-utils.h and probably a note should be added to UPDATING to run
makemandb with the -f flag to force recreation of the db.

--
Abhinav