Re: [patch] kbuild: add automatic updateconfig target

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 18 Jan 2006, Sam Ravnborg wrote:

> 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

ok will do.
 	
> > On Wed, Jan 18, 2006 at 08:40:56PM +0100, maximilian attems wrote:
> > > From: Bastian Blank <[email protected]>
> > > 
> > > 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.

ok easy stuff.
 
> > You did not introduce a specific update.config file like for the other
> > targets. Any reason for that?
> > 
> > 	Sam

naah, looks more like an oversight.
will rediff and resend.

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux