Re: pkg/29328: lintpkgsrc/pkg_info doesn't tell me which packages are missing +COMMENT

[Thomas Klausner <wiz%NetBSD.org@localhost>]

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

From: Thomas Klausner <wiz%NetBSD.org@localhost>
To: Timothy Lee Roden <diro%nixsys.bz@localhost>
Cc: gnats-bugs%netbsd.org@localhost
Subject: Re: pkg/29328: lintpkgsrc/pkg_info doesn't tell me which packages are 
missing +COMMENT
Date: Tue, 1 Mar 2005 14:20:25 +0100

 On Tue, Feb 22, 2005 at 10:24:02AM -0500, Timothy Lee Roden wrote:
 > I'm seeing a similar problem on NetBSD/alpha 1.6.2 with current-pkgsrc. This 
 > has been going on for a bit and I've not been able to determine the cause. 
 > It seems when certain pkgs are installed, +COMMENT is not entered into the 
 > pkg database. Also, on the daily security output, one may find:
 > 
 > < ruby18-rdtool-0.6.14 p5-XML-LibXML-1.57nb2 Perl interface to the libxml2
 > +library 
 > ---
 > > ruby18-rdtool-0.6.14 p5-perl-ldap-0.30nb2 Perl class to access LDAP servers
 > 1193c1192
 > 
 > Wherein (something similar):
 > 
 > % pkg_info -I ruby16
 > pkg_info: show_file: can't open '+COMMENT' for reading
 
 I guess this is caused by the /var/db/pkg/.refcount directory.
 It has since been moved to /var/db/pkg.refcount, but some binary
 packages might still include code that causes it to be regenerated.
 
 > ruby16-1.6.8nb1     
 > 
 > This behavior seems to have almost gone away. I once had almost 50 pkgs with 
 > unregistered COMMENTS, but now have it down to only one pkg with a missing 
 > entry.
 > 
 > If you update to pkgsrc-current, you may find this behavior to go away. 
 > However, this doesn't address the PR, which suggests lintpkgsrc/pkg_info 
 > should report which pkgs are missing a COMMENT. The request seems (to me at 
 > least) to be out of the jurisdiction of lintpkgsrc and as for pkg_info...the 
 > request seems useful to have, but shouldn't happen in the first place, as 
 > COMMENT should be written to the pkg database.
 
 This has been fixed in:
 revision 1.31
 date: 2005/02/16 08:35:26;  author: agc;  state: Exp;  lines: +4 -4
 Also display package name when having trouble displaying package
 information with show_index().
 ----------------------------
 revision 1.30
 date: 2005/02/14 16:16:24;  author: agc;  state: Exp;  lines: +5 -5
 Make show_file() take an extra argument, that of the package name, and
 display the package name with every warning message when displaying the
 contents of metadata files.
 
 Whilst I'm here, correct the function name in another warning message.
 ----------------------------
 
 of show/info.c.
 
 Cheers,
  Thomas