Introduce libxo to arp(8)

[garga]

Summary

Add libxo support to arp(8)

Sponsored by: Rubicon Communications (Netgate)
MFC after: 1 week

[garga]

Do not print hostname and ip address when it's not on arp table

[wblock]

usr.sbin/arp/arp.8

39

Does --libxo require options? Either way, this seems incomplete. Maybe

.Op Fl -libxo Ar options

or

.Op Fl -libxo Op Ar options

or something like that.

44

As above.

[garga]

This comment was removed by garga.

[garga]

usr.sbin/arp/arp.8

39

I've copied it from manpage of other binaries that already have libxo support. For instance netmap and df manpages are still like this in HEAD.

Should we add all possible libxo parameters on all binaries manpages?

[wblock]

usr.sbin/arp/arp.8

39

No, it is not necessary to list all the options, but we need to tell the user that an option is either expected or can be added here.

If other man pages do not say an option is either mandatory or optional here, they are wrong.

[garga]

Let user know --libxo parameter requires options as pointed by wblock

[wblock]

Man page looks okay to me. Please check with textproc/igor and mandoc -Tlint. Thanks!

[allanjude]

usr.sbin/arp/arp.c

179

need a xo_set_version() call in here before the container

[garga]

Added xo_set_version() call

[garga]

Make some changes as suggested by Phil Shafer <[e-mail]>

• Combine some xo_emit() calls
• Move hard coded strings to main string on some xo_emit() calls
• Change 'arp-table' to 'arp-cache'

[allanjude]

Approved By: allanjude

usr.sbin/arp/arp.8

31

update this to the current date before you commit