diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2008-03-27 16:34:54 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2008-04-23 16:13:41 -0400 |
commit | 1a448fdb3c5495405bc44d77ea676150f9195444 (patch) | |
tree | 77f5fea7c06a610e9913cf178e15a774e4f77380 /fs/Kconfig | |
parent | 3329ba05231808c96cf9fd0598b8f46afec9777c (diff) |
NFSD: Remove NFSv4 dependency on NFSv3
Clean up: Because NFSD_V4 "depends on" NFSD_V3, it appears as a child of
the NFSD_V3 menu entry, and is not visible if NFSD_V3 is unselected.
Replace the dependency on NFSD_V3 with a "select NFSD_V3". This makes
NFSD_V4 look and work just like NFS_V3, while ensuring that NFSD_V3 is
enabled if NFSD_V4 is.
Sam Ravnborg adds:
"This use of select is questionable. In general it is bad to select
a symbol with dependencies.
In this case the dependencies of NFSD_V3 are duplicated for NFSD_V4
so we will not se erratic configurations but do you remember to
update NFSD_V4 when you add a depends on NFSD_V3?
But I see no other clean way to do it right now."
Later he said:
"My comment was more to say we have things to address in kconfig.
This is abuse in the acceptable range."
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs/Kconfig')
-rw-r--r-- | fs/Kconfig | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/Kconfig b/fs/Kconfig index 157a88583b70..56c83f40cdbe 100644 --- a/fs/Kconfig +++ b/fs/Kconfig | |||
@@ -1755,7 +1755,8 @@ config NFSD_V3_ACL | |||
1755 | 1755 | ||
1756 | config NFSD_V4 | 1756 | config NFSD_V4 |
1757 | bool "NFS server support for NFS version 4 (EXPERIMENTAL)" | 1757 | bool "NFS server support for NFS version 4 (EXPERIMENTAL)" |
1758 | depends on NFSD && NFSD_V3 && PROC_FS && EXPERIMENTAL | 1758 | depends on NFSD && PROC_FS && EXPERIMENTAL |
1759 | select NFSD_V3 | ||
1759 | select FS_POSIX_ACL | 1760 | select FS_POSIX_ACL |
1760 | select RPCSEC_GSS_KRB5 | 1761 | select RPCSEC_GSS_KRB5 |
1761 | help | 1762 | help |