From patchwork Fri Aug 21 16:52:48 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steven Rostedt X-Patchwork-Id: 43129 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n7LGqqhf006164 for ; Fri, 21 Aug 2009 16:52:52 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751684AbZHUQwt (ORCPT ); Fri, 21 Aug 2009 12:52:49 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755287AbZHUQwt (ORCPT ); Fri, 21 Aug 2009 12:52:49 -0400 Received: from hrndva-omtalb.mail.rr.com ([71.74.56.125]:50212 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751684AbZHUQws (ORCPT ); Fri, 21 Aug 2009 12:52:48 -0400 Received: from gandalf ([74.67.89.75]) by hrndva-omta03.mail.rr.com with ESMTP id <20090821165249649.INEY2354@hrndva-omta03.mail.rr.com>; Fri, 21 Aug 2009 16:52:49 +0000 Date: Fri, 21 Aug 2009 12:52:48 -0400 (EDT) From: Steven Rostedt X-X-Sender: rostedt@gandalf.stny.rr.com To: LKML cc: Ingo Molnar , Andrew Morton , Thomas Gleixner , Peter Zijlstra , Linus Torvalds , Theodore Tso , Arjan van de Ven , Greg KH , "Luis R. Rodriguez" , zippel@linux-m68k.org, linux-kbuild@vger.kernel.org, Sam Ravnborg , Jonathan Corbet , Alan Jenkins , Andi Kleen Subject: [PATCH] kconfig: add missing dependency of conf to localyesconfig In-Reply-To: Message-ID: References: <20090819025634.987354707@goodmis.org> <20090821143525.GH11098@elte.hu> <20090821145406.GA3659@elte.hu> User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Sender: linux-kbuild-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kbuild@vger.kernel.org The following patch is in: git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-2.6-kconfig.git branch: for-next Steven Rostedt (1): kconfig: add missing dependency of conf to localyesconfig ---- scripts/kconfig/Makefile | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) --------------------------- commit 48586218b6515b9bd70694e3cd8c901a6a6ee69c Author: Steven Rostedt Date: Fri Aug 21 12:42:20 2009 -0400 kconfig: add missing dependency of conf to localyesconfig There's a dependency missing. $ make localyesconfig HOSTCC scripts/basic/fixdep HOSTCC scripts/basic/docproc HOSTCC scripts/basic/hash using config: '/boot/config-2.6.27.25-78.2.56.fc9.x86_64' /bin/sh: line 8: scripts/kconfig/conf: No such file or directory make[1]: *** [localyesconfig] Error 127 make: *** [localyesconfig] Error 2 Thus the script failed to run. But the sed command that converts the '=m' to '=y' still ran. This gives us a distro config with all modules converted to built in! The missing dependency was for conf for localyesconfig. This dependency was already set for localmodconfig. Reported-by: Ingo Molnar Signed-off-by: Steven Rostedt --- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile index 915a39a..6d69c7c 100644 --- a/scripts/kconfig/Makefile +++ b/scripts/kconfig/Makefile @@ -43,7 +43,7 @@ localmodconfig: $(obj)/streamline_config.pl $(obj)/conf fi $(Q)rm -f .tmp.config -localyesconfig: $(obj)/streamline_config.pl +localyesconfig: $(obj)/streamline_config.pl $(obj)/conf $(Q)perl $< $(Kconfig) > .tmp.config $(Q)sed -i s/=m/=y/ .tmp.config $(Q)if [ -f .config ]; then \