Re: [PATCH] configfs: change depends -> select SYSFS

[Nicholas A. Bellinger]

On Sun, 2011-01-16 at 09:54 -0800, Linus Torvalds wrote:
> On Sat, Jan 15, 2011 at 1:10 PM, Nicholas A. Bellinger
> <nab@xxxxxxxxxxxxxxx> wrote:
> >
> > This patch changes configfs to select SYSFS to fix the following:
> >
> > warning: (TARGET_CORE && GFS2_FS) selects CONFIGFS_FS which has unmet direct dependencies (SYSFS)
> 
> I'm ok with this part. But:
> 
> > This patch also fixes whitespace breakage in the 'help' section.
> 
> This is just bogus. The help text is _supposed_ to have that extra
> indentation so that it stands out from the rest of the Kconfig stuff.
> 

Ahh, thanks for the clarification here.   At some point I saw a patch
that did make this change, and it stuck with me that it was the
preferred method.  This won't happen again.

The updated patch to keep the extra indentation for configfs, along with
the three extra 'depends CONFIGFS_FS' patches from yesterday are
available in a pullable branch against this afternoon's
linux-2.6.git/master here:

git://git.kernel.org/pub/scm/linux/kernel/git/nab/scsi-post-merge-2.6.git for-linus

Please let me know if you have any questions,

--nab

Nicholas Bellinger (4):
  configfs: change depends -> select SYSFS
  net: Make NETCONSOLE_DYNAMIC depend on CONFIGFS_FS
  dlm: Make DLM depend on CONFIGFS_FS
  ocfs2: Make OCFS2_FS depend on CONFIGFS_FS

 drivers/net/Kconfig |    3 +--
 fs/configfs/Kconfig |    4 ++--
 fs/dlm/Kconfig      |    3 +--
 fs/ocfs2/Kconfig    |    3 +--
 4 files changed, 5 insertions(+), 8 deletions(-)


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/