From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:60013 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753934Ab1GLVWe (ORCPT ); Tue, 12 Jul 2011 17:22:34 -0400 Message-ID: <4E1CBB11.2090309@RedHat.com> Date: Tue, 12 Jul 2011 17:22:25 -0400 From: Steve Dickson To: Steve Dickson CC: "Gabor Z. Papp" , linux-nfs@vger.kernel.org Subject: Re: nfs-utils 1.2.4 References: <4E0E30A7.90209@RedHat.com> In-Reply-To: <4E0E30A7.90209@RedHat.com> Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-nfs-owner@vger.kernel.org List-ID: MIME-Version: 1.0 On 07/01/2011 04:40 PM, Steve Dickson wrote: > On 07/01/2011 02:36 AM, Gabor Z. Papp wrote: >> lo lo, >> >> trying to configure nfs-utils 1.2.4 with the options below: >> >> --disable-gss --disable-kprefix --enable-mount --enable-nfsv3 >> --disable-nfsv4 --with-statduser=nobody --with-statedir=/var/lib/nfs" >> --without-tcp-wrappers --disable-tirpc >> >> but getting the following error: >> >> [...] >> checking size of socklen_t... 4 >> configure: error: conditional "CONFIG_NFSIDMAP" was never defined. >> Usually this means the macro was only invoked conditionally. > The following patch seems to take care of the problem... The > problem stemmed from the use of the --disable-nfsv4, which > I will throw into my unit tests... Please let me know > if this fixes the problem... > > steved. > > > [PATCH] Make sure CONFIG_NFSIDMAP is always defined. > > CONFIG_NFSIDMAP always need to be define either > negatively or positive whether nfsv4 is or is not > defined. > > Signed-off-by: Steve Dickson > --- > configure.ac | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/configure.ac b/configure.ac > index ca12f9e..f8b0796 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -249,6 +249,9 @@ AC_CHECK_FUNC([getservbyname], , > > AC_CHECK_LIB([crypt], [crypt], [LIBCRYPT="-lcrypt"]) > > +dnl enable nfsidmap when its support by libnfsidmap > +AM_CONDITIONAL(CONFIG_NFSIDMAP, [test "$enable_nfsidmap" = "yes"]) > + > if test "$enable_nfsv4" = yes; then > dnl check for libevent libraries and headers > AC_LIBEVENT > @@ -256,9 +259,6 @@ if test "$enable_nfsv4" = yes; then > dnl check for nfsidmap libraries and headers > AC_LIBNFSIDMAP > > - dnl enable nfsidmap when its support by libnfsidmap > - AM_CONDITIONAL(CONFIG_NFSIDMAP, [test "$enable_nfsidmap" = "yes"]) > - > dnl check for the keyutils libraries and headers > AC_KEYUTILS > Committed.... steved.