Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:31525 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933112AbeARQRt (ORCPT ); Thu, 18 Jan 2018 11:17:49 -0500 Message-ID: <1516292263.3594.16.camel@redhat.com> Subject: Re: [PATCH] Remove unneeded reference to nfsidmap_internal.h From: Justin Mitchell To: Chuck Lever Cc: Steve Dickson , Linux NFS Mailing List Date: Thu, 18 Jan 2018 16:17:43 +0000 In-Reply-To: References: <20180117221020.11056.59627.stgit@manet.1015granger.net> <1516267701.3594.11.camel@redhat.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-nfs-owner@vger.kernel.org List-ID: On Thu, 2018-01-18 at 09:26 -0500, Chuck Lever wrote: > > > On Jan 18, 2018, at 4:28 AM, Justin Mitchell wrote: > > > > On Wed, 2018-01-17 at 17:10 -0500, Chuck Lever wrote: > >> This file is not part of "git clone" and is not #included from any > >> C file. > >> > >> Signed-off-by: Chuck Lever > >> --- > >> support/nfsidmap/Makefile.am | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> Hey Steve- > >> > >> I don't see nfsidmap_internal.h anywhere. Wondering if I missed > >> something? > > > > It was removed as part of my ABI cleanup (commit f080188e) to stop > > libnfsidmap from leaking all sorts of private symbols. > > > > Some of it went to libnfsidmap_plugin.h and some to > > libnfsidmap_private.h > > > > Some external packages which offered libnfsidmap plugins were using > > these undocumented symbols by copying libnfsidmap_internal.h, so i > > reworked the neccessary parts into an API as libnfsidmap_plugin.h and > > hid the rest. > > > > I clearly missed this reference to the file as it didn't generate any > > warnings for me, sorry about that. > > No apologies needed. Are you confirming that this clean up is OK? > Can I get an amen? (I mean, can I get a Reviewed-by?) > > > -- > Chuck Lever > Yes, all looks good. thanks Reviewed-by: Justin Mitchell