From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754510Ab1HJR7c (ORCPT ); Wed, 10 Aug 2011 13:59:32 -0400 Received: from casper.infradead.org ([85.118.1.10]:38310 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753703Ab1HJR7b (ORCPT ); Wed, 10 Aug 2011 13:59:31 -0400 Subject: Re: [PATCH 2/2] Enable 'make CONFIG_FOO=y oldconfig' From: David Woodhouse To: Arnaud Lacombe Cc: Michal Marek , "H. Peter Anvin" , linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org Date: Wed, 10 Aug 2011 18:59:27 +0100 In-Reply-To: References: <1312067499.22074.59.camel@i7.infradead.org> <1312067670.22074.61.camel@i7.infradead.org> <4E34998D.8060806@zytor.com> <4E4140B3.7090802@suse.cz> <4E4280BA.9050109@suse.cz> <1312981658.2989.39.camel@i7.infradead.org> <1312985888.2989.46.camel@i7.infradead.org> <1312993987.2989.53.camel@i7.infradead.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.0.2 (3.0.2-3.fc15) Content-Transfer-Encoding: 7bit Message-ID: <1312999168.2989.61.camel@i7.infradead.org> Mime-Version: 1.0 X-SRS-Rewrite: SMTP reverse-path rewritten from by casper.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2011-08-10 at 13:44 -0400, Arnaud Lacombe wrote: > Exactly my point, you just successfully created an half-backed config > which is different than what Aunt Tillie wanted you to generate. This > should be an hard error, same for "all*.config", not to mention that > the error message is far from being helpful. You are whining about something that has been true of the kernel config system for at least the last 16 years that I've been working on it, You're *right*, of course, but you're getting on my tits by whining about it only *now*, in this context. At least I have offered *an* error message reporting that the request was not honoured, which is a whole lot better that we've been used to. Please, if this offends you then by all means go and fix it. A sane way of handling dependencies would give a way to say "do what you need to do in order to enable CONFIG_SATA_MV", and should remove the abomination of 'select', which was introduced purely to work around that lack. But none of that is directly relevant in *this* thread. -- dwmw2