[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20061120153528.5ace6089.randy.dunlap@oracle.com>
Date: Mon, 20 Nov 2006 15:35:28 -0800
From: Randy Dunlap <randy.dunlap@...cle.com>
To: Adrian Bunk <bunk@...sta.de>
Cc: Stefan Richter <stefanr@...6.in-berlin.de>,
Patrick Caulfield <pcaulfie@...hat.com>,
cluster-devel@...hat.com, linux-kernel@...r.kernel.org,
David Teigland <teigland@...hat.com>,
Andrew Morton <akpm@...l.org>
Subject: Re: [PATCH 2.6.19-rc5-mm2] fs/dlm: fix recursive dependency in
Kconfig
On Mon, 20 Nov 2006 18:45:09 +0100 Adrian Bunk wrote:
> On Mon, Nov 20, 2006 at 12:45:11PM +0100, Stefan Richter wrote:
> >...
> > Anyway. Whatever you chose to do (or already have chosen to do) in
> > fs/dlm/Kconfig, keep in mind that the "select" keyword is presently only
> > poorly supported by the various .config generators and that it forces UI
> > considerations into the Kconfig files which should better not be
> > overloaded with UI issues. Or in other words: It is rather easy to write
> > correct and well-supported Kconfig files if you stick with "depend on",
> > but you get into trouble fast with generous usage of "select".
>
> For variables like NET or INET it doesn't matter in practice whether you
> use "select" or "depends on". But for other variables it makes it really
> hard for users to enable an option if you use "depends on".
Doing a "select" NET or INET enables a ton of code, which IMO
should be done explicitly, not covertly by a select.
---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists