On Thu, Nov 11, 2021 at 07:01:42AM +0100, Bjorn Ketelaars wrote: > On Wed 10/11/2021 21:20, Klemens Nanni wrote: > > I think only unwind(8) should list all the inputs and unwindctl(8) > > should just say "Show learned nameservers". > > > > unwind(8) is already incomplete regardless of sppp(4) and unwindctl(8) > > is a poor duplicate of it. > > I agree with unwindctl(8). For unwind(8) i would like to take it one > step further and use resolvd(8) as single source of truth. What do you > think of the diff below?
I'm not convinced: users shouldn't read manuals of tools they don't use in order to learn how the tools they do use are working. > diff --git sbin/unwind/unwind.8 sbin/unwind/unwind.8 > index afa7d0ad2c1..38f7d1aaea8 100644 > --- sbin/unwind/unwind.8 > +++ sbin/unwind/unwind.8 > @@ -33,11 +33,8 @@ It is intended to run on client machines like workstations > or laptops and only > listens on localhost. > .Nm > sends DNS queries to nameservers to answer queries and switches to resolvers > -learned from > -.Xr dhclient 8 , > -.Xr dhcpleased 8 > -or > -.Xr slaacd 8 > +learned by > +.Xr resolvd 8 > if it detects that DNS queries are blocked by the local network. > It periodically probes if DNS is no longer blocked and switches back to > querying nameservers itself. This is wrong. Stop resolvd and start unwind, you'll see that `unwindctl status autoconf' shows learned nameservers. > @@ -104,9 +101,7 @@ socket used for communication with > .El > .Sh SEE ALSO > .Xr unwind.conf 5 , > -.Xr dhclient 8 , > -.Xr dhcpleased 8 , > -.Xr slaacd 8 , > +.Xr resolvd 8 , > .Xr unbound 8 , > .Xr unwindctl 8 > .Sh STANDARDS > diff --git usr.sbin/unwindctl/unwindctl.8 usr.sbin/unwindctl/unwindctl.8 > index 1b1be4451e7..b8289a82bd9 100644 > --- usr.sbin/unwindctl/unwindctl.8 > +++ usr.sbin/unwindctl/unwindctl.8 > @@ -56,11 +56,7 @@ Reload the configuration file. > .It Cm status > Show a status summary. > .It Cm status autoconf > -Show nameservers learned from > -.Xr dhclient 8 , > -.Xr dhcpleased 8 > -or > -.Xr slaacd 8 . > +Show learned nameservers. > .It Cm status memory > Show memory consumption. > .El That's what I have in my WIP diff already.
