On Fri, May 31, 2019 at 10:20:16PM +0100, Larry Hynes wrote:
> Hi
> 

hi.

> - Use "user's password" (instead of "users password") consistently

agreed

> - Change "to be read something like" to "to something like"

i think it was probably intended to be "to read something like", which i
think reads better than your proposal.

> - Change "Be sure reset" to "Be sure to reset" (I'm not sure if line
>   length is an issue here, so I deleted the arguably superfluous "back"
>   from "reset these vales back", in case it is)
> 

agreed

> Following is a patch for etc.amd64/login.conf against -current. If/when
> there is agreement on the proposed changes, I will send patches for
> other archs.
> 

if you can update per second comment and do for all archs, it's ok me.
if no one else picks it up, i'll commit myself.

jmc

> Index: login.conf
> ===================================================================
> RCS file: /cvs/src/etc/etc.amd64/login.conf,v
> retrieving revision 1.10
> diff -u -p -r1.10 login.conf
> --- login.conf        2 May 2019 08:35:10 -0000       1.10
> +++ login.conf        31 May 2019 21:18:56 -0000
> @@ -8,7 +8,7 @@
>  # Standard authentication styles:
>  #
>  # passwd     Use only the local password file
> -# chpass     Do not authenticate, but change users password (change
> +# chpass     Do not authenticate, but change user's password (change
>  #            the YP password if the user has one, else change the
>  #            local password)
>  # lchpass    Do not login; change user's local password instead
> @@ -33,7 +33,7 @@ auth-ftp-defaults:auth-ftp=passwd:
>  # The default values
>  # To alter the default authentication types change the line:
>  #    :tc=auth-defaults:\
> -# to be read something like: (enables passwd, "myauth", and activ)
> +# to something like: (enables passwd, "myauth", and activ)
>  #    :auth=passwd,myauth,activ:\
>  # Any value changed in the daemon class should be reset in default
>  # class.
> @@ -55,7 +55,7 @@ default:\
>  #
>  # Settings used by /etc/rc and root
>  # This must be set properly for daemons started as root by inetd as well.
> -# Be sure reset these values back to system defaults in the default class!
> +# Be sure to reset these values to system defaults in the default class!
>  #
>  daemon:\
>       :ignorenologin:\
> 

Reply via email to