diff mbox

nfs-utils 1.2.4

Message ID 4E0E30A7.90209@RedHat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Steve Dickson July 1, 2011, 8:40 p.m. UTC
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(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Gabor Z. Papp July 4, 2011, 6:39 a.m. UTC | #1
* 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.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Steve Dickson July 12, 2011, 9:22 p.m. UTC | #2
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.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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