Re: pkg/58131: empty +REQUIRED_BY

[Stefan Schaeckeler <schaecsn%gmx.com@localhost>]

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

From: Stefan Schaeckeler <schaecsn%gmx.com@localhost>
To: gnats-bugs%netbsd.org@localhost, pkg-manager%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
 pkgsrc-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/58131: empty +REQUIRED_BY
Date: Tue, 23 Apr 2024 22:37:23 -0700

 >  * On 2024-04-10 at 17:44 BST, schaecsn%gmx.com@localhost wrote:
 >
 >  >I have quite a lot of empty +REQUIRED_BY files. That confuses pkg_leav=
 e
 >  >(and perhaps other utilities). How come?
 >
 >  Hard to say without knowing how you got into that state, but needless t=
 o
 >  say it's not a correct state to be in!
 >
 >  You can fix it up by running "pkg_admin rebuild-tree".
 
 
 Hmm, now I know why pkg_leaves did not work.
 
 
 - - - snip - - -
 # pkg_info  -Q automatic gcc10-aux-10.3.0nb1
 YES
 - - - snip - - -
 
 
 the check in pkg_leaves is against lower case "yes":
 
 - - - snip - - -
 		if [ ${skip_manual} =3D "yes" ]; then
                         if [ "`${PKG_INFO} -K "${PKG_DBDIR}" -Q automatic =
 ${pkg} 2> /dev/null`" !=3D yes ]; then
 - - - snip - - -
 
 
 Having YES capitalized was a user error. Sorry.
 
 - - - snip - - -
 /var/db/pkg$ grep automatic */+INSTALLED_INFO
 SDL-1.2.15nb43/+INSTALLED_INFO:automatic=3Dyes
 SDL_net-1.2.8nb1/+INSTALLED_INFO:automatic=3Dyes
 TECkit-2.5.12/+INSTALLED_INFO:automatic=3Dyes
 abcm2ps-8.13.12nb11/+INSTALLED_INFO:automatic=3DYES
 ...
 - - - snip - - -
 
 
 So, the empty +REQUIRED_BY was not the fault why pkg_leave did not work. T=
 hat's now a feature request, though. Having empty +REQUIRED_BY files delet=
 ed makes inspecting /var/db/pkg/*/ directories more straightforward.
 
  Stefan