svn commit: r224674 - head/etc

[Garrett Cooper]

On Sun, Aug 7, 2011 at 9:12 PM, Doug Barton <dougb at freebsd.org> wrote:
> On 8/7/2011 6:05 PM, Garrett Cooper wrote:
>> On Sun, Aug 7, 2011 at 1:51 PM, Simon L. B. Nielsen <simon at nitro.dk> wrote:
>>>
>>> On 6 Aug 2011, at 11:16, Doug Barton wrote:
>>>
>>>> Author: dougb
>>>> Date: Sat Aug  6 09:16:53 2011
>>>> New Revision: 224674
>>>> URL: http://svn.freebsd.org/changeset/base/224674
>>>>
>>>> Log:
>>>>  Rather than edit the nsswitch.conf file based on ${MK_NIS} == "no"
>>>>  comment out the NIS _compat options by default, but leave them in
>>>>  the file for the convenience of users who want to enable it.
>>>>
>>>>  Update the comment in the file accordingly.
>>>
>>> This disables NIS by default which is rather surprising considering there has been no prior (which I have been able to find) discussion of this.
>>>
>>> At the _VERY_ least people need to be warned about this loudly. Unless you can show this really helps non-NIS users this is IMO a bad idea.
>>>
>>> Yes, if you hand merge all /etc files this won't be a problem... but I don't since life's to short for that, so this change just broke login on one of the FreeBSD.org during an upgrade.
>
> mergemaster has been in the tree for almost 12 years now. :)  In any
> case, I can't think of a situation where blindly copying new files to
> /etc is a good idea (with all due respect).
>
>> non-POLA changes are bad, and this _definitely_ deserves an UPDATING
>> entry and/or further discussion.
>
> I'm happy to ask for permission to do the UPDATING entry, do you have
> suggestions for the text?

Maybe this?
-Garrett

$ svn diff UPDATING
Index: UPDATING
===================================================================
--- UPDATING    (revision 224699)
+++ UPDATING    (working copy)
@@ -22,6 +22,10 @@
        machines to maximize performance.  (To disable malloc debugging, run
        ln -s aj /etc/malloc.conf.)

+20110807:
+       Prior versions of nsswitch.conf explicitly enabled NIS lookup of group,
+       passwd, and services information; it is now disabled by default.
+
 20110628:
        The packet filter (pf) code has been updated to OpenBSD 4.5.
        You need to update userland tools to be in sync with kernel.