public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: Patrick Caulfield <pcaulfie@redhat.com>,
	cluster-devel@redhat.com, linux-kernel@vger.kernel.org,
	David Teigland <teigland@redhat.com>,
	Andrew Morton <akpm@osdl.org>
Subject: Re: [PATCH 2.6.19-rc5-mm2] fs/dlm: fix recursive dependency in Kconfig
Date: Mon, 20 Nov 2006 18:45:09 +0100	[thread overview]
Message-ID: <20061120174509.GW31879@stusta.de> (raw)
In-Reply-To: <45619547.5070301@s5r6.in-berlin.de>

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".

> Stefan Richter

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


  reply	other threads:[~2006-11-20 17:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-18 15:52 [PATCH 2.6.19-rc5-mm2] fs/dlm: fix recursive dependency in Kconfig Stefan Richter
2006-11-20  9:48 ` Patrick Caulfield
2006-11-20 11:45   ` Stefan Richter
2006-11-20 17:45     ` Adrian Bunk [this message]
2006-11-20 23:35       ` Randy Dunlap
2006-11-21  0:12         ` Adrian Bunk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20061120174509.GW31879@stusta.de \
    --to=bunk@stusta.de \
    --cc=akpm@osdl.org \
    --cc=cluster-devel@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pcaulfie@redhat.com \
    --cc=stefanr@s5r6.in-berlin.de \
    --cc=teigland@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox