cxgbetool(8): cleanup

[yuri_aetern.org]

Summary

• use Bl -column instead of TS which allows us to use macros inside the table cells
• fix a lot of other issues reported by mandoc -Tlint

Test Plan

mandoc -Tlint (empty now)

[pauamma_gundo.com]

Patched manual page now renders.

Changes other than to tables affect markup only, not text. (I can't tell for changes to tables.)

[yuri_aetern.org]

Adding np@ as it's somewhat "external" man page, want to be sure it won't break the process for vendor.

[freebsd_igalic.co]

This man page is still broken.
@np can you take a look at this review, please.

[np]

Sorry I missed this previously. This man page has a dependency on groff and I'd like to get rid of it eventually (the dependency, not the man page). But my manpage-fu is weak and I'm not sure how to figure out exactly which parts need groff. Will these proposed changes make any difference as far as reliance on groff goes? Okay to check this in either way.

man cxgbetool

This manpage needs groff(1) to be rendered
First install groff(1):
pkg install groff

[danfe]

@np wrote:

But my manpage-fu is weak and I'm not sure how to figure out exactly which parts need groff.

I believe it's those table tags:

_
ipv4    T{
.Cm type ipv4
T}      T{
incoming packet is an IPv4 datagram.
T}

Need to convert them to .Bl/.It+Ta/.El, but we'd better ask someone from the manpages to be sure.

[debdrup]

In D37881#899585, @danfe wrote:

Need to convert them to .Bl/.It+Ta/.El, but we'd better ask someone from the manpages to be sure.

Yes, these are the correct macros to use for lists.

If the submitter wants to fix it, I'll be happy to commit it.