On Tue, Apr 30, 2024 at 02:50:58PM +0200, Solene Rapenne wrote:
> this diff adds a new check in daily(8) using rcctl ls rogue
> 
> maybe wording could be merged better with the current text
> to avoid repeating rc.conf.local(8) twice.
> 
> ok?
> 

hi.

i think the natural way to say this would be "and vice versa":

	Lists any daemons which are enabled in
	.Xr rc.conf.local 8
	but which are not actually running
	(and vice versa).

jmc

> Index: share/man/man8/daily.8
> ===================================================================
> RCS file: /cvs/src/share/man/man8/daily.8,v
> diff -u -p -r1.29 daily.8
> --- share/man/man8/daily.8	20 Oct 2020 22:42:29 -0000	1.29
> +++ share/man/man8/daily.8	30 Apr 2024 12:49:59 -0000
> @@ -112,7 +112,9 @@ for the mount options, e.g.
>  Checks daemon status.
>  Lists any daemons which are enabled in
>  .Xr rc.conf.local 8
> -but which are not actually running.
> +but which are not actually running, and any running daemons not
> +enabled in
> +.Xr rc.conf.local 8 .
>  .It
>  Reports on which file systems need to be dumped via
>  .Xr dump 8 .
> Index: etc/daily
> ===================================================================
> RCS file: /cvs/src/etc/daily,v
> diff -u -p -r1.97 daily
> --- etc/daily	3 Mar 2023 16:22:57 -0000	1.97
> +++ etc/daily	30 Apr 2024 12:49:59 -0000
> @@ -136,6 +136,9 @@ done
>  next_part "Services that should be running but aren't:"
>  rcctl ls failed
>  
> +next_part "Services that are running but shouldn't:"
> +rcctl ls rogue
> +
>  next_part "Filesystems which need to be dumped:"
>  dump w | grep -vB1 ^Dump
>  
>