From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: [PATCH] configfs: change depends -> select SYSFS Date: Sun, 16 Jan 2011 17:07:38 -0800 Message-ID: <4D33965A.1090505@oracle.com> References: <1295125851-25279-1-git-send-email-nab@linux-iscsi.org> <20110116141105.59e5b3a2@stein> <1295214807.22813.57.camel@haakon2.linux-iscsi.org> <20110117000659.57352da7@stein> <20110116231901.GF22723@ZenIV.linux.org.uk> <20110117015220.11b43ae7@stein> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Linus Torvalds , Al Viro , "Nicholas A. Bellinger" , linux-kernel , linux-scsi , linux-fsdevel , Joel Becker , James Bottomley To: Stefan Richter Return-path: In-Reply-To: <20110117015220.11b43ae7@stein> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On 01/16/11 16:52, Stefan Richter wrote: > On Jan 16 Linus Torvalds wrote: >> Or do you expect a high-quality implementation of a configuration >> script to allow the user to just say "I want ocfs2", and then figure >> out the dependencies and solve them for you? >> >> I'd say that the latter case is OBVIOUSLY the quality implementation, >> while the former one is just stupid. > > As Al noted, there also needs to be a convenient way to *disable* options. > > Hence, the Kconfig files should merely declare the dependencies. Then, > some user interfaces should deal with the this database of dependencies > and let different people achieve different goals as easily as possible. > > We already have different UIs. But perhaps we have too much UI down in > the Kconfig files. I.e., perhaps Kconfig shouldn't know the difference > between depends-on and select. Let the different UIs present the > dependency graph to the operator in different ways, suitable to different > tasks. I agree, I think that the UIs should list a kconfig symbol and its dependencies, then let the user enable them as needed. > /handwave off. -- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***