From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 81154 invoked by alias); 29 Aug 2017 20:23:30 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 81129 invoked by uid 89); 29 Aug 2017 20:23:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.0 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_PASS,URIBL_RED autolearn=ham version=3.3.2 spammy=H*c:HHHHH X-HELO: relay1.mentorg.com Date: Tue, 29 Aug 2017 20:23:00 -0000 From: Joseph Myers To: Steve Ellcey CC: DJ Delorie , Subject: Re: [PATCH] Remove compat from DEFAULT_CONFIG lookup strings In-Reply-To: <1504033460.5204.8.camel@cavium.com> Message-ID: References: <1503951778.28672.139.camel@cavium.com> <1503957036.28672.144.camel@cavium.com> <1504033460.5204.8.camel@cavium.com> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="-1152306461-1134611247-1504038199=:23565" X-ClientProxiedBy: svr-ies-mbx-01.mgc.mentorg.com (139.181.222.1) To svr-ies-mbx-01.mgc.mentorg.com (139.181.222.1) X-SW-Source: 2017-08/txt/msg01260.txt.bz2 ---1152306461-1134611247-1504038199=:23565 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Content-length: 958 On Tue, 29 Aug 2017, Steve Ellcey wrote: > No reason, I just didn't realise libnss_nis wasn't built by default. > > Here is an updated patch for setting the default lookup methods in nss. > I updated nss/nsswitch.c to not use nis lookups when RPC is not > configured and I updated DEFAULT_CONFIG in grp/initgroups.c to be > "compat [NOTFOUND=return] files" when using NSL instead of just "compat > files".  DJ noticed that and I think the missing NOTFOUND is just an > oversight and it conflicted with the documentation in manual/nss.texi. > Fixing the code seemed more reasonable than documenting the existing > setting.  I did update the documentation to reflect the behavour of > this patch. > > I tested once with "--enable-obsolete-nsl --enable-obsolete-rpc" and > once without and had no regressions from the unmodified glibc on > aarch64. > > OK to checkin? I have no further comments on this patch. -- Joseph S. Myers joseph@codesourcery.com ---1152306461-1134611247-1504038199=:23565--