Stuart Henderson <s...@spacehopper.org> wrote:

> : Index: share/man/man4/wg.4
> : ===================================================================
> : RCS file: /cvs/src/share/man/man4/wg.4,v
> : retrieving revision 1.10
> : diff -u -p -u -r1.10 wg.4
> : --- share/man/man4/wg.4     14 Mar 2021 10:08:38 -0000      1.10
> : +++ share/man/man4/wg.4     24 Dec 2022 00:49:05 -0000
> : @@ -42,6 +42,19 @@ configuration file for
> :  .Xr netstart 8 .
> :  The interface itself can be configured with
> :  .Xr ifconfig 8 .
> : +To display
> : +.Cm wgpeer
> : +information for each
> : +.Nm wg
> : +interface option
> : +.Fl A
> : +to
> : +.Xr ifconfig 8
> : +should be used or
> : +.Nm wg
> : +interface should be specified as an argument to
> : +.Xr ifconfig 8
> : +command.
> :  .Pp
> :  .Nm wg
> :  interfaces support the following
> 
> This isn't directly related to the descr diff so should be a separate
> commit if done.
> 
> This diff displays like so:
> 
> -----
>      The interfaces can be created at runtime using the ifconfig wgN create
>      command or by setting up a hostname.if(5) configuration file for
>      netstart(8).  The interface itself can be configured with ifconfig(8).
>      To display wgpeer information for each wg interface option -A to
>      ifconfig(8) should be used or wg interface should be specified as an
>      argument to ifconfig(8) command.
>      [...]
> -----
> 
> I find this new text a bit confusing really, and really it's more
> general to ifconfig rather than specific to wg, so I'm not sure how
> much explanation is really needed in wg(4). How about just adding
> a couple of lines to Examples instead?
> 
> Index: wg.4
> ===================================================================
> RCS file: /cvs/src/share/man/man4/wg.4,v
> retrieving revision 1.10
> diff -u -p -r1.10 wg.4
> --- wg.4      14 Mar 2021 10:08:38 -0000      1.10
> +++ wg.4      14 Jan 2023 14:24:58 -0000
> @@ -164,6 +164,11 @@ which resides in the default
>  Show the listening sockets:
>  .Pp
>  .Dl $ netstat -ln
> +.Pp
> +Show full information for peers on a single interface, or all interfaces:
> +.Pp
> +.Dl # ifconfig wg1
> +.Dl # ifconfig -A
>  .Sh DIAGNOSTICS
>  The
>  .Nm

Or just skip all this text explaining how to use ifconfig, because people
need to learn how to use ifconfig from the ifconfig manual page.

Reply via email to