On Mon, Feb 11, 2008 at 02:31:37PM +0100, Sam Ravnborg wrote: > Hi Linus. > > A kbuild bug sneaked in. Please pull from: > > ssh://master.kernel.org/pub/scm/linux/kernel/git/sam/kbuild.git > > to fix the following bug. > > Sam
Mike spotted another missing thing from his initial patch so I folded it into the fix and pushed out a new kbuild.git tree. See updated patch below. Sam >From 49af821e4b1c07e756cbc2e389eba9d885912602 Mon Sep 17 00:00:00 2001 From: Sam Ravnborg <[EMAIL PROTECTED]> Date: Mon, 11 Feb 2008 14:26:26 +0100 Subject: [PATCH] kbuild: fix make V=1 When make -s support were added to filechk to combination created with make V=1 were not covered. Fix it by explicitly cover this case too. Signed-off-by: Sam Ravnborg <[EMAIL PROTECTED]> Cc: Mike Frysinger <[EMAIL PROTECTED]> --- scripts/Kbuild.include | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/init/Makefile b/init/Makefile index c5f157c..4a243df 100644 --- a/init/Makefile +++ b/init/Makefile @@ -27,6 +27,7 @@ $(obj)/version.o: include/linux/compile.h # mkcompile_h will make sure to only update the # actual file if its content has changed. + chk_compile.h = : quiet_chk_compile.h = echo ' CHK $@' silent_chk_compile.h = : include/linux/compile.h: FORCE diff --git a/scripts/Kbuild.include b/scripts/Kbuild.include index da3559e..d64e6ba 100644 --- a/scripts/Kbuild.include +++ b/scripts/Kbuild.include @@ -39,10 +39,13 @@ escsq = $(subst $(squote),'\$(squote)',$1) # - If they are equal no change, and no timestamp update # - stdin is piped in from the first prerequisite ($<) so one has # to specify a valid file as first prerequisite (often the kbuild file) + chk_filechk = : quiet_chk_filechk = echo ' CHK $@' silent_chk_filechk = : + upd_filechk = : quiet_upd_filechk = echo ' UPD $@' silent_upd_filechk = : + define filechk $(Q)set -e; \ $($(quiet)chk_filechk); \ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/