On Tue, Sep 22, 2020 at 11:15:08AM +0200, m...@fn.de wrote: > > On 2020-09-22 10:26, Stuart Henderson wrote: > > > > This has put me off the idea further tbh. Once committed it can't simply > > be ignored, if it's not maintained in a timely fashion if things change > > in OpenBSD (which is not that unusual) then somebody else has to either > > fix or remove it. This happened many times before for ports which are > > OpenBSD-specific. > > > Unlikely that it would be unmaintained since it is not my intention to give > up on using OpenBSD. So I will either adapt or remove if not adaptable. > However, I understand your concern. Just wanted to simplify for those who > would like to have it.> > > Since it does sound different to show-reverse-deps (though it wasn't > > clear from the description), > > True, the description is not referring to and not comparing with > show-reverse-deps, but it is clearly saying what it does and what it > doesn't. > > > perhaps the solution is to add the > > functionality as a new mode in pkg_info. > > > Maybe later. > > Look, we already have all the code to compute dependency closures for installed packages.
The only thing your code might do is show a dag/tree of it, which might be cool, or not. I will say it once again: do not grep directly inside /var/db/pkg do not grep packing-lists to extract info. That's not just a question of "unsupported" it's a question that it might fail in non-obvious ways. We've seen that situation before, with whatever outside tool was showing us an incomplete repository of all ports, because it didn't actually know the semantics of flavors and multi-packages, and was trying to roll its own. That's the main issue there: if I do change internals of packages, it may break your tool in subtle ways... in most cases, I am the person go to whenever something feels strange in pkgland/ports land. So your tool, in effect, is a maintenance burden for me. I have *nothing* against presenting the information you present in a nice way. But *do* go through the actual mandated API to get it.