On Wed, Nov 04, 2015 at 03:14:56AM +0000, Raf Czlonka wrote:
> Hi all,
> 
> Some explanation regarding the changes:
> 
> - might -> "needs to" - it has to be there, no "might" about it :^)
> - I've added the additional httpd_flags example back - without it, the
>   following sentence doesn't make sense. Alternatively the "As can be
>   seen..." sentence can be removed entirely.
> - even though "package" hadn't, most likely, been used literally the
>   words means has a special meaning in OpenBSD, which may lead to some
>   confusion
> - since the DNS is being used in brackets, let's expand it properly
> - named -> nsd has been missed when unbound's been added to the man page
> - since eigrpd appeared recently, it's a good excuse to add it here
> - "being running" doesn't sound good
> 
> Regards,
> 
> Raf
> 
> Index: share/man/man8/intro.8
> ===================================================================
> RCS file: /cvs/src/share/man/man8/intro.8,v
> retrieving revision 1.26
> diff -u -p -r1.26 intro.8
> --- share/man/man8/intro.8    21 Sep 2015 09:23:35 -0000      1.26
> +++ share/man/man8/intro.8    4 Nov 2015 02:23:45 -0000
> @@ -90,11 +90,14 @@ file overrides variable assignments in
>  .Xr rc.conf 8 .
>  So to enable
>  .Xr httpd 8 ,
> -the following line might be added to
> +the following line needs to be added to

this is not really correct. the "might" indicates this is one way to
enable httpd. as you show yourself immediately below, there are other
ways.

>  .Pa /etc/rc.conf.local :
>  .Bd -literal -offset indent
>  httpd_flags=""
>  .Ed
> +or
> +.Bd -literal -offset indent -compact
> +httpd_flags="-vv"

without your initial change, this would be unneccessary.

>  .Pp
>  As can be seen above,
>  this method is also used to specify additional options.
> @@ -106,7 +109,7 @@ and the individual pages for the utiliti
>  .Ss Automounter daemon (amd)
>  If using the
>  .Xr amd 8
> -package,
> +daemon,

yes, i agree this is a better choice of words.

>  go into the
>  .Pa /etc/amd
>  directory and set it up by
> @@ -123,7 +126,7 @@ variables:
>  .Va amd_flags
>  and
>  .Va amd_master .
> -.Ss Domain Name Server (DNS)
> +.Ss Domain Name System (DNS)

that reads better in one way, but worse in another. i honestly think
it's fine as-is. we could change it to "DNS server" i guess. or servers.

>  If you are using a name server, check the
>  .Pa /etc/resolv.conf
>  file.
> @@ -141,7 +144,7 @@ If using a caching name server, add the 
>  Relevant
>  .Xr rc.conf 8
>  variables:
> -.Va named_flags
> +.Va nsd_flags
>  and
>  .Va unbound_flags .
>  .Ss Clock synchronisation
> @@ -224,6 +227,7 @@ variables:
>  .Ss Routing Daemons
>  Various daemons for managing routing tables are available:
>  .Xr bgpd 8 ,
> +.Xr eigrpd 8 ,
>  .Xr ospfd 8 ,
>  and
>  .Xr ripd 8 .
> @@ -232,13 +236,14 @@ Relevant
>  .Xr rc.conf 8
>  variables:
>  .Va bgpd_flags ,
> +.Va eigrpd_flags ,
>  .Va ospfd_flags ,
>  and
>  .Va ripd_flags .
>  .Ss RPC-based network services
>  Several services depend on the RPC portmapper,
>  .Xr portmap 8 ,
> -being running for proper operation.
> +running, for proper operation.

"being running" is correct. i agree it's a difficult concept though. but
without it, you can read it as several services depend on rpc (running
for proper operation), where the part i marked in brackets is "one
thing".

>  This includes YP and NFS exports, among other services.
>  .Pp
>  Relevant
>

jmc

Reply via email to