On Wed, Aug 31, 2022 at 12:02:20PM +0200, Claudio Jeker wrote:
> On Wed, Aug 31, 2022 at 08:37:06AM +0000, Klemens Nanni wrote:
> > On Fri, Aug 26, 2022 at 06:14:44PM +0000, Klemens Nanni wrote:
> > > Naming the list like the struct itself makes for awful grepping.
> > > Distinguish the list name; no functional change.
> > >
> > > Builds/runs fine on and64 and sparc64.
> > >
> >
> > Ping.
> >
> > deraadt pointed out how base programs used to use this symbol, e.g.
> >
> > struct nlist nl[] = {
> > { "ifnet" },
> > ...
> > };
> >
> > Nothing in base does this anymore, but some ports still do:
> > https://codesearch.debian.net/search?q=filetype%3Ac+%22ifnet%22&literal=1
> >
> > net/libgtop2
> > net/libnet
> > net/net-snmp
> > net/nmap
> >
> > Similar to the upcoming REVISION bump due to struct ifnet's
> > if_switchport member removal of some ports, I can go through the above
> > list and follow-up on the rename.
>
> I know that such REVISION bumps are cheap but to be honest kvm readers are
> known the break frequently and since kvm is disabled by default it is
> questionable if these tools are still usable (also it is probably some old
> compat code that you find in codesearch.debian.net).
Yes, I did not audit every port in question whether it's actually used
on OpenBSD, but, as you said, bumps are cheap and at the very least they
do document possible offenders in RCS history.
I'd waste much more time inspecting every port, so better bump them all
and be done with it.
>
> --
> :wq Claudio
>
