linux-nfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* nfs-utils 1.2.4
@ 2011-07-01  6:36 Gabor Z. Papp
  2011-07-01 20:40 ` Steve Dickson
  0 siblings, 1 reply; 4+ messages in thread
From: Gabor Z. Papp @ 2011-07-01  6:36 UTC (permalink / raw)
  To: linux-nfs

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.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: nfs-utils 1.2.4
  2011-07-01  6:36 nfs-utils 1.2.4 Gabor Z. Papp
@ 2011-07-01 20:40 ` Steve Dickson
  2011-07-04  6:39   ` Gabor Z. Papp
  2011-07-12 21:22   ` Steve Dickson
  0 siblings, 2 replies; 4+ messages in thread
From: Steve Dickson @ 2011-07-01 20:40 UTC (permalink / raw)
  To: Gabor Z. Papp; +Cc: linux-nfs

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 <steved@redhat.com>
---
 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
 

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: nfs-utils 1.2.4
  2011-07-01 20:40 ` Steve Dickson
@ 2011-07-04  6:39   ` Gabor Z. Papp
  2011-07-12 21:22   ` Steve Dickson
  1 sibling, 0 replies; 4+ messages in thread
From: Gabor Z. Papp @ 2011-07-04  6:39 UTC (permalink / raw)
  To: Steve Dickson; +Cc: linux-nfs

* Steve Dickson <SteveD@redhat.com>:

| 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... 

| [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.

Yes, problem fixed. Thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: nfs-utils 1.2.4
  2011-07-01 20:40 ` Steve Dickson
  2011-07-04  6:39   ` Gabor Z. Papp
@ 2011-07-12 21:22   ` Steve Dickson
  1 sibling, 0 replies; 4+ messages in thread
From: Steve Dickson @ 2011-07-12 21:22 UTC (permalink / raw)
  To: Steve Dickson; +Cc: Gabor Z. Papp, linux-nfs



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 <steved@redhat.com>
> ---
>  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.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-07-12 21:22 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-01  6:36 nfs-utils 1.2.4 Gabor Z. Papp
2011-07-01 20:40 ` Steve Dickson
2011-07-04  6:39   ` Gabor Z. Papp
2011-07-12 21:22   ` Steve Dickson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).