Re: [patch] kbuild: add automatic updateconfig target

From: Sam Ravnborg
Date: Wed Jan 18 2006 - 15:46:33 EST


On Wed, Jan 18, 2006 at 09:42:34PM +0100, Sam Ravnborg wrote:
> Please always add Roman Zippel when dealing with kconfig changes.
And I should do so in my replies - yes!
Please use this mail for futher comments to keep all in the loop.

Sam


> On Wed, Jan 18, 2006 at 08:40:56PM +0100, maximilian attems wrote:
> > From: Bastian Blank <waldi@xxxxxxxxxx>
> >
> > current hack for daily build linux-2.6-git is quite ugly:
> > yes "n" | make oldconfig
> >
> > belows target helps to build git snapshots in a more automated way,
> > setting the new options to their default.
>
> Please always add Roman Zippel when dealing with kconfig changes.
> We had a similar though more advanced proposal named miniconfig a month
> or so ago but Roman had some grief with it so it was not applied.
>
> I've let Roman decide on this one too.
> Nitpicking below.
>
> Sam
>
>
> > +updateconfig: $(obj)/conf
> > + $< -U arch/$(ARCH)/Kconfig
>
> The other methods uses small letters so please change to '-u'
>
> > - set_random
> > + set_random,
> > + update_default,
>
> Keep same naming as the others. May I suggest set_default.
>
> You did not introduce a specific update.config file like for the other
> targets. Any reason for that?
>
> Sam
> -
> 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-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/