On Monday, July 05, 2010, Michal Marek wrote: > On Mon, Jul 05, 2010 at 11:25:56PM +0200, Rafael J. Wysocki wrote: > > Hmm. Am I dong anything particularly wrong, or is CONFIG_LOCALVERSION ignored > > now, as it seems? > > (also reported by Stephen earlier that day) > > On Mon, Jul 05, 2010 at 10:39:41AM +1000, Stephen Rothwell wrote: > > After merging the libata tree, today's linux-next build produced this > > warning (but it should be a failure ...): > > > > /bin/sh: scripts/setlocalversion: No such file or directory > > > > Probably introduced by commit 0a564b2645c8766a669c55bde1f1ef5b0518caec > > ("kbuild: Propagate LOCALVERSION= down to scripts/setlocalversion"). > > > > That commit really isn't ready for inclusion in Linus' tree ... > > > Oops, I'm an idiot. The below patch should fix localversion again: Yes, it helps here. Thanks, Rafael > From: Michal Marek <mmarek@xxxxxxx> > Subject: [PATCH] kbuild: Fix path to scripts/setlocalversion > > Commit 0a564b2 broke LOCALVERSION for O=... builds. Ouch. > > Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> > Reported-by: Rafael J. Wysocki <rjw@xxxxxxx> > Signed-off-by: Michal Marek <mmarek@xxxxxxx> > > diff --git a/Makefile b/Makefile > index 12ab175..df16c4f 100644 > --- a/Makefile > +++ b/Makefile > @@ -886,7 +886,7 @@ $(vmlinux-dirs): prepare scripts > # Store (new) KERNELRELASE string in include/config/kernel.release > include/config/kernel.release: include/config/auto.conf FORCE > $(Q)rm -f $@ > - $(Q)echo "$(KERNELVERSION)$$($(CONFIG_SHELL) scripts/setlocalversion $(srctree))" > $@ > + $(Q)echo "$(KERNELVERSION)$$($(CONFIG_SHELL) $(srctree)/scripts/setlocalversion $(srctree))" > $@ > > > # Things we need to do before we recursively start building the kernel > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > > -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html