Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965251AbWHWW27 (ORCPT ); Wed, 23 Aug 2006 18:28:59 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965246AbWHWW27 (ORCPT ); Wed, 23 Aug 2006 18:28:59 -0400 Received: from xenotime.net ([66.160.160.81]:43910 "HELO xenotime.net") by vger.kernel.org with SMTP id S965251AbWHWW26 (ORCPT ); Wed, 23 Aug 2006 18:28:58 -0400 Date: Wed, 23 Aug 2006 15:32:07 -0700 From: "Randy.Dunlap" To: Matt Helsley Cc: Dave Hansen , devel@openvz.org, Andrew Morton , Rik van Riel , Chandra Seetharaman , Greg KH , Linux Kernel Mailing List , Andi Kleen , Christoph Hellwig , Andrey Savochkin , Alan Cox , Rohit Seth , Oleg Nesterov Subject: Re: [Devel] [PATCH 1/6] BC: kconfig Message-Id: <20060823153207.0019aa3f.rdunlap@xenotime.net> In-Reply-To: <1156371222.2510.715.camel@stark> References: <44EC31FB.2050002@sw.ru> <44EC35A3.7070308@sw.ru> <1156370698.12011.55.camel@localhost.localdomain> <1156371222.2510.715.camel@stark> Organization: YPO4 X-Mailer: Sylpheed version 2.2.7 (GTK+ 2.8.10; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1827 Lines: 49 On Wed, 23 Aug 2006 15:13:42 -0700 Matt Helsley wrote: > On Wed, 2006-08-23 at 15:04 -0700, Dave Hansen wrote: > > On Wed, 2006-08-23 at 15:01 +0400, Kirill Korotaev wrote: > > > --- ./arch/sparc64/Kconfig.arkcfg 2006-07-17 17:01:11.000000000 +0400 > > > +++ ./arch/sparc64/Kconfig 2006-08-10 17:56:36.000000000 +0400 > > > @@ -432,3 +432,5 @@ source "security/Kconfig" > > > source "crypto/Kconfig" > > > > > > source "lib/Kconfig" > > > + > > > +source "kernel/bc/Kconfig" > > ... > > > --- ./arch/sparc64/Kconfig.arkcfg 2006-07-17 17:01:11.000000000 +0400 > > > +++ ./arch/sparc64/Kconfig 2006-08-10 17:56:36.000000000 +0400 > > > @@ -432,3 +432,5 @@ source "security/Kconfig" > > > source "crypto/Kconfig" > > > > > > source "lib/Kconfig" > > > + > > > +source "kernel/bc/Kconfig" > > > > Is it just me, or do these patches look a little funky? Looks like it > > is trying to patch the same thing into the same file, twice. Also, the > > patches look to be -p0 instead of -p1. > > They do appear to be -p0 > > They aren't adding the same thing twice to the same file. This patch > makes different arches source the same Kconfig. Look again. There are 2 diffstat blocks and 2 of these at least: --- ./kernel/bc/Kconfig.bckm 2006-07-28 13:07:38.000000000 +0400 +++ ./kernel/bc/Kconfig 2006-07-28 13:09:51.000000000 +0400 @@ -0,0 +1,25 @@ > I seem to recall Chandra suggested that instead of doing it this way it > would be more appropriate to add the source line to init/Kconfig because > it's more central and arch-independent. I tend to agree. --- ~Randy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/