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

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

The following reply was made to PR bin/46419; it has been noted by GNATS.

From: Abhinav Upadhyay <er.abhinav.upadhyay%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost
Subject: Re: bin/46419: Inconsistent information in man.db for man page
 aliases after updation
Date: Mon, 7 May 2012 12:23:11 +0530

 On Mon, May 7, 2012 at 6:15 AM, Matthew Mondor 
<mm_lists%pulsar-zone.net@localhost> w=
 rote:
 > 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
 > =A0aliases after updation
 > Date: Sun, 6 May 2012 20:44:57 -0400
 >
 > =A0On Sun, =A06 May 2012 19:00:01 +0000 (UTC)
 > =A0er.abhinav.upadhyay%gmail.com@localhost wrote:
 >
 > =A0> 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 d=
 atabase. 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.
 >
 > =A0Sorry that I didn't yet read the whole implementation, thus this
 > =A0question:
 >
 > =A0I see that a new column is added to the new table, which gets created
 > =A0if it doesn't exist. =A0Is this table deleted and recreated at every
 > =A0makemandb run? =A0Otherwise, some migration code might be needed to al=
 ter
 > =A0the existing table and add the new column (or to delete the existing
 > =A0tables if they're not of the intended version/format)...
 >
 > =A0Or perhaps this doesn't matter as there was no official release yet
 > =A0with the new code, and UPDATING could hold a note with instructions fo=
 r
 > =A0the admins to delete the old db (if that is a problem, that is)?
 >
 > =A0Thanks,
 > =A0--
 
 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