Brad,

-----Original Message-----
From: <owner-src-committ...@freebsd.org> on behalf of Brad Davis 
<b...@freebsd.org>
Date: 2018-08-15, Wednesday at 09:22
To: <src-committ...@freebsd.org>, <svn-src-...@freebsd.org>, 
<svn-src-head@freebsd.org>
Subject: svn commit: r337857 - in head: bin/csh bin/sh etc

> Author: brd
> Date: Wed Aug 15 16:22:12 2018
> New Revision: 337857
> URL: https://svnweb.freebsd.org/changeset/base/337857
> 
> Log:
>   Fix build after r337849
>   
>   This moves the symlink creation to after where the files are installed.
>   
>   This also inverts the shell change so that it only happens if MK_TCSH is on.
...
> Modified: head/etc/master.passwd
> ==============================================================================
> --- head/etc/master.passwd    Wed Aug 15 16:16:59 2018        (r337856)
> +++ head/etc/master.passwd    Wed Aug 15 16:22:12 2018        (r337857)
> @@ -1,6 +1,6 @@
>  # $FreeBSD$
>  #
> -root::0:0::0:0:Charlie &:/root:/bin/csh
> +root::0:0::0:0:Charlie &:/root:/bin/sh
>  toor:*:0:0::0:0:Bourne-again Superuser:/root:
>  daemon:*:1:1::0:0:Owner of many system processes:/root:/usr/sbin/nologin
>  operator:*:2:5::0:0:System &:/:/usr/sbin/nologin

Woah! Changing the root shell wasn't mentioned in the change description, has 
nothing to do with fixing r337849, and is a *HUGE* POLA violation. At the very 
least, a change of this magnitude needs public discussion, and even if the 
community agreed, it would also require an UPDATING message and relnote.

Please revert this change to master.passwd immediately.

-Ravi (rpokala@)


_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to