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

next in thread | previous in thread | raw e-mail | index | archive | help
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 =A06 09:16:53 2011
>> New Revision: 224674
>> URL: http://svn.freebsd.org/changeset/base/224674
>>
>> Log:
>> =A0Rather than edit the nsswitch.conf file based on ${MK_NIS} =3D=3D "no=
"
>> =A0comment out the NIS _compat options by default, but leave them in
>> =A0the file for the convenience of users who want to enable it.
>>
>> =A0Update 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 yo=
u 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 do=
n't since life's to short for that, so this change just broke login on one =
of the FreeBSD.org during an upgrade.

non-POLA changes are bad, and this _definitely_ deserves an UPDATING
entry and/or further discussion.
Thanks,
-Garrett



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