[PATCH 2.5.62]: 3/3: Very small menu cleanup

[PATCH 2.5.62]: 3/3: Very small menu cleanup

[Marc-Christian Petersen]

[-- Attachment #1: Type: text/plain, Size: 230 bytes --]


1. Move "JBD (ext3) debugging support" two spaces rightwards

2. Can we please rename "Second extended fs support" to
   "Ext2 fs support" to be conform with:

	Ext2 extended attributes
	  Ext2 POSIX Access Control Lists

[-- Attachment #2: menu-fixes.pach --]
[-- Type: text/x-diff, Size: 760 bytes --]

diff -Naurp linux-2.5.62-vanilla/fs/Kconfig linux-2.5.62-mcp1/fs/Kconfig
--- linux-2.5.62-vanilla/fs/Kconfig	2003-02-10 19:38:43.000000000 +0100
+++ linux-2.5.62-mcp1/fs/Kconfig	2003-02-18 13:43:48.000000000 +0100
@@ -336,7 +336,7 @@ config JBD
 	  you cannot compile this code as a module.
 
 config JBD_DEBUG
-	bool "JBD (ext3) debugging support"
+	bool "  JBD (ext3) debugging support"
 	depends on JBD
 	---help---
 	  If you are using the ext3 journaled file system (or potentially any
@@ -943,7 +943,7 @@ config ROMFS_FS
 	  answer N.
 
 config EXT2_FS
-	tristate "Second extended fs support"
+	tristate "Ext2 fs support"
 	---help---
 	  This is the de facto standard Linux file system (method to organize
 	  files on a storage device) for hard disks.

Re: [PATCH 2.5.62]: 3/3: Very small menu cleanup

[Roman Zippel]

Hi,

On Tue, 18 Feb 2003, Marc-Christian Petersen wrote:

> 1. Move "JBD (ext3) debugging support" two spaces rightwards

Don't do this, indentation is now derived from the dependencies. Adding 
spaces helps 'make config' but not really the other front ends.
The correct fix would be to change JBD into:

config JBD
	bool
	default y
	depends on EXT3_FS

Now the following JBD_DEBUG entry should be an entry under EXT3_FS,
but unfortunately there seems to be bug somewhere, it's inserted to high 
in the menu tree. As a work around you can add EXT3_FS as dependency to 
JBD_DEBUG, I'll look into it as soon as possible.

bye, Roman