Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 7 Aug 2011 23:42:33 -0700
From:      Garrett Cooper <yanegomi@gmail.com>
To:        Doug Barton <dougb@freebsd.org>
Cc:        svn-src-head@freebsd.org, "Simon L. B. Nielsen" <simon@nitro.dk>, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r224674 - head/etc
Message-ID:  <CAGH67wQ6vcgvRn2F_VaLmSausUdaaasCmvboJVx4OXaAG_gyvg@mail.gmail.com>
In-Reply-To: <4E3F6216.5080706@FreeBSD.org>
References:  <201108060916.p769Gr4A043462@svn.freebsd.org> <9DDF0DAB-9056-45CD-8CE9-81B621A35B13@nitro.dk> <CAGH67wQ1NNrT8becXq5oFgXf7QP6ZUmqLEiaBqdO-afJBx4K2w@mail.gmail.com> <4E3F6216.5080706@FreeBSD.org>

index | next in thread | previous in thread | raw e-mail

On Sun, Aug 7, 2011 at 9:12 PM, Doug Barton <dougb@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@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.


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGH67wQ6vcgvRn2F_VaLmSausUdaaasCmvboJVx4OXaAG_gyvg>