Re: [PATCH] kbuild: fix oldnoconfig to do the right thing

From: Michal Marek
Date: Wed Sep 08 2010 - 07:56:14 EST


On Sun, Sep 05, 2010 at 08:17:58AM +0200, Sam Ravnborg wrote:
> On Wed, Sep 01, 2010 at 11:21:30AM -0400, Kyle McMartin wrote:
> > From: Kyle McMartin <kyle@xxxxxxxxxx>
> >
> > Commit 861b4ea4 broke oldnoconfig when removed the oldnoconfig checks on
> > if (input_mode == nonint_oldconfig ||
> > input_mode == oldnoconfig) {
> > if (input_mode == nonint_oldconfig &&
> > sym->name &&
> > !sym_is_choice_value(sym)) {
> > to avoid oldnoconfig chugging through the else stanza.
> >
> > Fix that to restore expected behaviour (which I've confirmed in the
> > Fedora kernel build that the configs end up looking the same.)
>
> I played aroud with this.
> I looks like the only difference I got was that before
> your patch oldnoconfig produced a lot of output to stdout.
> With your patch is was silent.
>
> The resulting config was the same.

I found that when running make oldnoconfig on the kconfig branch of
kbuild.git (which is currently a 2.6.35 + kconfig changes) against a
2.6.35-rc1 x86_64 defconfig, it mysteriously switches from
CONFIG_KERNEL_GZIP=y to CONFIG_KERNEL_LZO=y. With Kyle's patch it
doesn't do this. I applied the patch now.

Michal
--
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/