From: Caspar Schutijser <caspar@schutijser.com>
Subject: Re: NEW: net/dnsviz
To: ports@openbsd.org
Date: Mon, 9 Jun 2025 16:51:12 +0200

Hi,

I think it would be great to have dnsviz packaged! Some feedback below.

On Sat, May 24, 2025 at 03:32:02AM +0000, Lucas Gabriel Vuotto wrote:
> Attached are both net/dnsviz and the a new dep,
> math/py-pygraphviz. Do mind, that's not math/py-graphviz; is there
> something that can be done about it?

The upstream project is called pygraphviz so I guess that's just the
way it is, even it looks a little bit confusing. So I'd leave it as is.
That's just my opinion though.

For net/dnsviz, portcheck says:
> manual pages should go under ${PREFIX}/man/ rather than under ${PREFIX}/share/man/
Can you fix that?

Nit: for the licensing comment, most ports just write "GPLv2" in this
case.

> Tarball includes both ports, to be
> extracted on /usr/ports.
> 
> Information for inst:dnsviz-0.11.1
> 
> Comment:
> tool suite for analysis and visualitzation of DNS

There's a typo in this comment for net/dnsviz. And generally I'd reword
it, perhaps something like "tool suite for analyzing and visualizing
the DNS". Potentially add " and DNSSEC" if there's room. And then apply
the same changes to pkg/DESCR.

> Description:
> DNSViz is a tool suite for analysis and visualization of Domain Name System
> (DNS) behavior, including its security extensions (DNSSEC). This tool suite
> powers the Web-based analysis available at https://dnsviz.net/
             ^ I'd suggest a lower-case 'w'

> Information for inst:py3-pygraphviz-1.14
> 
> Comment:
> Python interface to Graphviz graph drawing package

Suggestion:
Python interface to the Graphviz visualization package


Can you fix the above points? I tested it on amd64, works well there.

Caspar